On Wed, Nov 07, 2012 at 04:52:50PM +, Liu, Jinsong wrote:
> Konrad Rzeszutek Wilk wrote:
> > On Wed, Nov 07, 2012 at 12:58:19PM +, Liu, Jinsong wrote:
> >
> > If it is generic ACPI code, than it can depend only on ACPI.
> > If it is ACPI code that contains X86 specific info,
Konrad Rzeszutek Wilk wrote:
> On Wed, Nov 07, 2012 at 12:58:19PM +, Liu, Jinsong wrote:
>
> If it is generic ACPI code, than it can depend only on ACPI.
> If it is ACPI code that contains X86 specific info, than it needs
> to depend on X86 too.
No x86 specific so
On Wed, Nov 07, 2012 at 12:58:19PM +, Liu, Jinsong wrote:
> >>>
> >>> If it is generic ACPI code, than it can depend only on ACPI.
> >>> If it is ACPI code that contains X86 specific info, than it needs to
> >>> depend on X86 too.
> >>
> >> No x86 specific so let's depend on ACPI.
> >
> >
>>>
>>> If it is generic ACPI code, than it can depend only on ACPI.
>>> If it is ACPI code that contains X86 specific info, than it needs to
>>> depend on X86 too.
>>
>> No x86 specific so let's depend on ACPI.
>
> Huh? This feature is x86 specific isn't it? I mean it is in the ACPI
> spec,
If it is generic ACPI code, than it can depend only on ACPI.
If it is ACPI code that contains X86 specific info, than it needs to
depend on X86 too.
No x86 specific so let's depend on ACPI.
Huh? This feature is x86 specific isn't it? I mean it is in the ACPI
spec, but only x86 does it
On Wed, Nov 07, 2012 at 12:58:19PM +, Liu, Jinsong wrote:
If it is generic ACPI code, than it can depend only on ACPI.
If it is ACPI code that contains X86 specific info, than it needs to
depend on X86 too.
No x86 specific so let's depend on ACPI.
Huh? This feature is x86
Konrad Rzeszutek Wilk wrote:
On Wed, Nov 07, 2012 at 12:58:19PM +, Liu, Jinsong wrote:
If it is generic ACPI code, than it can depend only on ACPI.
If it is ACPI code that contains X86 specific info, than it needs
to depend on X86 too.
No x86 specific so let's depend on ACPI.
Huh?
On Wed, Nov 07, 2012 at 04:52:50PM +, Liu, Jinsong wrote:
Konrad Rzeszutek Wilk wrote:
On Wed, Nov 07, 2012 at 12:58:19PM +, Liu, Jinsong wrote:
If it is generic ACPI code, than it can depend only on ACPI.
If it is ACPI code that contains X86 specific info, than it needs
to
On Tue, Nov 06, 2012 at 04:23:57PM +, Liu, Jinsong wrote:
> Stefano Stabellini wrote:
> > On Tue, 6 Nov 2012, Liu, Jinsong wrote:
> >> Liu, Jinsong wrote:
> >>> Stefano Stabellini wrote:
> >> diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
> >> index 0e86370..a2af622 100644
>
Stefano Stabellini wrote:
> On Tue, 6 Nov 2012, Liu, Jinsong wrote:
>> Liu, Jinsong wrote:
>>> Stefano Stabellini wrote:
>> diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
>> index 0e86370..a2af622 100644
>> --- a/drivers/xen/Makefile
>> +++ b/drivers/xen/Makefile
>>
On Tue, 6 Nov 2012, Liu, Jinsong wrote:
> Liu, Jinsong wrote:
> > Stefano Stabellini wrote:
> diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
> index 0e86370..a2af622 100644
> --- a/drivers/xen/Makefile
> +++ b/drivers/xen/Makefile
> @@ -29,6 +29,7 @@
On Tue, 6 Nov 2012, Liu, Jinsong wrote:
Liu, Jinsong wrote:
Stefano Stabellini wrote:
diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
index 0e86370..a2af622 100644
--- a/drivers/xen/Makefile
+++ b/drivers/xen/Makefile
@@ -29,6 +29,7 @@ obj-$(CONFIG_XEN_MCE_LOG) +=
Stefano Stabellini wrote:
On Tue, 6 Nov 2012, Liu, Jinsong wrote:
Liu, Jinsong wrote:
Stefano Stabellini wrote:
diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
index 0e86370..a2af622 100644
--- a/drivers/xen/Makefile
+++ b/drivers/xen/Makefile
@@ -29,6 +29,7 @@
On Tue, Nov 06, 2012 at 04:23:57PM +, Liu, Jinsong wrote:
Stefano Stabellini wrote:
On Tue, 6 Nov 2012, Liu, Jinsong wrote:
Liu, Jinsong wrote:
Stefano Stabellini wrote:
diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
index 0e86370..a2af622 100644
---
Liu, Jinsong wrote:
> Stefano Stabellini wrote:
diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
index 0e86370..a2af622 100644
--- a/drivers/xen/Makefile
+++ b/drivers/xen/Makefile
@@ -29,6 +29,7 @@ obj-$(CONFIG_XEN_MCE_LOG) += mcelog.o
Stefano Stabellini wrote:
>>> diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
>>> index 0e86370..a2af622 100644
>>> --- a/drivers/xen/Makefile
>>> +++ b/drivers/xen/Makefile
>>> @@ -29,6 +29,7 @@ obj-$(CONFIG_XEN_MCE_LOG) += mcelog.o
>>> obj-$(CONFIG_XEN_PCIDEV_BACKEND) +=
> > diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
> > index 0e86370..a2af622 100644
> > --- a/drivers/xen/Makefile
> > +++ b/drivers/xen/Makefile
> > @@ -29,6 +29,7 @@ obj-$(CONFIG_XEN_MCE_LOG) += mcelog.o
> > obj-$(CONFIG_XEN_PCIDEV_BACKEND) += xen-pciback/
> >
>>> On 03.11.12 at 13:16, "Liu, Jinsong" wrote:
> Konrad Rzeszutek Wilk wrote:
From f514b97628945cfac00efb0d456f133d44754c9d Mon Sep 17 00:00:00
2001
>>> From: Liu, Jinsong
>>> Date: Thu, 1 Nov 2012 21:02:36 +0800
>>> Subject: [PATCH 1/2] Xen acpi pad implement
>>>
>>> PAD is acpi
On 03.11.12 at 13:16, Liu, Jinsong jinsong@intel.com wrote:
Konrad Rzeszutek Wilk wrote:
From f514b97628945cfac00efb0d456f133d44754c9d Mon Sep 17 00:00:00
2001
From: Liu, Jinsong jinsong@intel.com
Date: Thu, 1 Nov 2012 21:02:36 +0800
Subject: [PATCH 1/2] Xen acpi pad implement
diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
index 0e86370..a2af622 100644
--- a/drivers/xen/Makefile
+++ b/drivers/xen/Makefile
@@ -29,6 +29,7 @@ obj-$(CONFIG_XEN_MCE_LOG) += mcelog.o
obj-$(CONFIG_XEN_PCIDEV_BACKEND) += xen-pciback/
Stefano Stabellini wrote:
diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
index 0e86370..a2af622 100644
--- a/drivers/xen/Makefile
+++ b/drivers/xen/Makefile
@@ -29,6 +29,7 @@ obj-$(CONFIG_XEN_MCE_LOG) += mcelog.o
obj-$(CONFIG_XEN_PCIDEV_BACKEND) += xen-pciback/
Liu, Jinsong wrote:
Stefano Stabellini wrote:
diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile
index 0e86370..a2af622 100644
--- a/drivers/xen/Makefile
+++ b/drivers/xen/Makefile
@@ -29,6 +29,7 @@ obj-$(CONFIG_XEN_MCE_LOG) += mcelog.o
obj-$(CONFIG_XEN_PCIDEV_BACKEND)
Konrad Rzeszutek Wilk wrote:
>>> From f514b97628945cfac00efb0d456f133d44754c9d Mon Sep 17 00:00:00
>>> 2001
>> From: Liu, Jinsong
>> Date: Thu, 1 Nov 2012 21:02:36 +0800
>> Subject: [PATCH 1/2] Xen acpi pad implement
>>
>> PAD is acpi Processor Aggregator Device which provides a control
>>
Konrad Rzeszutek Wilk wrote:
From f514b97628945cfac00efb0d456f133d44754c9d Mon Sep 17 00:00:00
2001
From: Liu, Jinsong jinsong@intel.com
Date: Thu, 1 Nov 2012 21:02:36 +0800
Subject: [PATCH 1/2] Xen acpi pad implement
PAD is acpi Processor Aggregator Device which provides a control
On Thu, Nov 01, 2012 at 06:34:45AM +, Liu, Jinsong wrote:
> Thanks! updated as attached.
>
> Jinsong
>
> =
> >From f514b97628945cfac00efb0d456f133d44754c9d Mon Sep 17 00:00:00 2001
> From: Liu, Jinsong
> Date: Thu, 1 Nov 2012 21:02:36 +0800
> Subject: [PATCH 1/2] Xen
On Thu, Nov 01, 2012 at 06:34:45AM +, Liu, Jinsong wrote:
Thanks! updated as attached.
Jinsong
=
From f514b97628945cfac00efb0d456f133d44754c9d Mon Sep 17 00:00:00 2001
From: Liu, Jinsong jinsong@intel.com
Date: Thu, 1 Nov 2012 21:02:36 +0800
Subject: [PATCH
Thanks! updated as attached.
Jinsong
=
>From f514b97628945cfac00efb0d456f133d44754c9d Mon Sep 17 00:00:00 2001
From: Liu, Jinsong
Date: Thu, 1 Nov 2012 21:02:36 +0800
Subject: [PATCH 1/2] Xen acpi pad implement
PAD is acpi Processor Aggregator Device which provides a
Thanks! updated as attached.
Jinsong
=
From f514b97628945cfac00efb0d456f133d44754c9d Mon Sep 17 00:00:00 2001
From: Liu, Jinsong jinsong@intel.com
Date: Thu, 1 Nov 2012 21:02:36 +0800
Subject: [PATCH 1/2] Xen acpi pad implement
PAD is acpi Processor Aggregator Device
Konrad Rzeszutek Wilk wrote:
> On Tue, Oct 30, 2012 at 03:18:59PM +, Liu, Jinsong wrote:
>> Konrad Rzeszutek Wilk wrote:
> +config XEN_ACPI_PAD_STUB
> + bool
> + depends on XEN_DOM0 && X86_64 && ACPI
> + default n
> +
This Kconfig is pointless, if
On Tue, Oct 30, 2012 at 03:18:59PM +, Liu, Jinsong wrote:
> Konrad Rzeszutek Wilk wrote:
> >>> +config XEN_ACPI_PAD_STUB
> >>> + bool
> >>> + depends on XEN_DOM0 && X86_64 && ACPI
> >>> + default n
> >>> +
> >>
> >> This Kconfig is pointless, if CONFIG_XEN_ACPI_PAD_STUB = n, native
> >> pad
On Tue, Oct 30, 2012 at 03:18:59PM +, Liu, Jinsong wrote:
Konrad Rzeszutek Wilk wrote:
+config XEN_ACPI_PAD_STUB
+ bool
+ depends on XEN_DOM0 X86_64 ACPI
+ default n
+
This Kconfig is pointless, if CONFIG_XEN_ACPI_PAD_STUB = n, native
pad would successfully registerred, and
Konrad Rzeszutek Wilk wrote:
On Tue, Oct 30, 2012 at 03:18:59PM +, Liu, Jinsong wrote:
Konrad Rzeszutek Wilk wrote:
+config XEN_ACPI_PAD_STUB
+ bool
+ depends on XEN_DOM0 X86_64 ACPI
+ default n
+
This Kconfig is pointless, if CONFIG_XEN_ACPI_PAD_STUB = n, native
pad would
Konrad Rzeszutek Wilk wrote:
>>> +config XEN_ACPI_PAD_STUB
>>> + bool
>>> + depends on XEN_DOM0 && X86_64 && ACPI
>>> + default n
>>> +
>>
>> This Kconfig is pointless, if CONFIG_XEN_ACPI_PAD_STUB = n, native
>> pad would successfully registerred, and then mwait #UD (we would
>> revert
> > +config XEN_ACPI_PAD_STUB
> > + bool
> > + depends on XEN_DOM0 && X86_64 && ACPI
> > + default n
> > +
>
> This Kconfig is pointless, if CONFIG_XEN_ACPI_PAD_STUB = n, native pad would
> successfully registerred, and then mwait #UD (we would revert
>
Konrad Rzeszutek Wilk wrote:
> On Mon, Oct 29, 2012 at 03:38:14AM +, Liu, Jinsong wrote:
>> It's doable to register a stub for xen. However, it's not preferred,
>> say, to make xen pad as module, considering the case 'rmmod
>> xen_acpi_pad' then 'insmod acpi_pad'? Under such case there is risk
Konrad Rzeszutek Wilk wrote:
On Mon, Oct 29, 2012 at 03:38:14AM +, Liu, Jinsong wrote:
It's doable to register a stub for xen. However, it's not preferred,
say, to make xen pad as module, considering the case 'rmmod
xen_acpi_pad' then 'insmod acpi_pad'? Under such case there is risk
of
+config XEN_ACPI_PAD_STUB
+ bool
+ depends on XEN_DOM0 X86_64 ACPI
+ default n
+
This Kconfig is pointless, if CONFIG_XEN_ACPI_PAD_STUB = n, native pad would
successfully registerred, and then mwait #UD (we would revert
df88b2d96e36d9a9e325bfcd12eb45671cbbc937, right?). So
Konrad Rzeszutek Wilk wrote:
+config XEN_ACPI_PAD_STUB
+ bool
+ depends on XEN_DOM0 X86_64 ACPI
+ default n
+
This Kconfig is pointless, if CONFIG_XEN_ACPI_PAD_STUB = n, native
pad would successfully registerred, and then mwait #UD (we would
revert
On Mon, Oct 29, 2012 at 03:38:14AM +, Liu, Jinsong wrote:
> It's doable to register a stub for xen. However, it's not preferred, say, to
> make xen pad as module, considering the case 'rmmod xen_acpi_pad' then
> 'insmod acpi_pad'? Under such case there is risk of mwait #UD, if we want to
>
On Mon, Oct 29, 2012 at 03:38:14AM +, Liu, Jinsong wrote:
It's doable to register a stub for xen. However, it's not preferred, say, to
make xen pad as module, considering the case 'rmmod xen_acpi_pad' then
'insmod acpi_pad'? Under such case there is risk of mwait #UD, if we want to
It's doable to register a stub for xen. However, it's not preferred, say, to
make xen pad as module, considering the case 'rmmod xen_acpi_pad' then 'insmod
acpi_pad'? Under such case there is risk of mwait #UD, if we want to remove
mwait check as 'patch 2/2:
It's doable to register a stub for xen. However, it's not preferred, say, to
make xen pad as module, considering the case 'rmmod xen_acpi_pad' then 'insmod
acpi_pad'? Under such case there is risk of mwait #UD, if we want to remove
mwait check as 'patch 2/2:
On Fri, Oct 26, 2012 at 02:00:36PM +, Liu, Jinsong wrote:
> Updated per Jan's comments.
>
> Thanks,
> Jinsong
>
> ===
>
> >From e6d4eceba9538c6873f9c970425e65e257fd9bf0 Mon Sep 17 00:00:00 2001
> From: Liu, Jinsong
> Date: Sat, 27 Oct 2012 05:34:50 +0800
> Subject: [PATCH 1/2]
Updated per Jan's comments.
Thanks,
Jinsong
===
>From e6d4eceba9538c6873f9c970425e65e257fd9bf0 Mon Sep 17 00:00:00 2001
From: Liu, Jinsong
Date: Sat, 27 Oct 2012 05:34:50 +0800
Subject: [PATCH 1/2] Xen acpi pad implement
PAD is acpi Processor Aggregator Device which provides a
>>> On 25.10.12 at 14:19, "Liu, Jinsong" wrote:
> --- /dev/null
> +++ b/drivers/xen/xen_acpi_pad.c
> @@ -0,0 +1,173 @@
> +/*
> + * xen_acpi_pad.c - Xen pad interface
> + *
> + * Copyright (c) 2012, Intel Corporation.
> + *Author: Liu, Jinsong
> + *
> + * This program is free software; you
] [PATCH 1/2] Xen acpi pad implement
>>> "Liu, Jinsong" 10/26/12 8:18 AM >>>
>> +static struct acpi_driver xen_acpi_pad_driver = {
>> +.name = "processor_aggregator",
>> +.class = ACPI_PROCESSOR_AGGREGATOR_CLASS,
>>
>>> "Liu, Jinsong" 10/26/12 8:18 AM >>>
>> +static struct acpi_driver xen_acpi_pad_driver = {
>> +.name = "processor_aggregator",
>> +.class = ACPI_PROCESSOR_AGGREGATOR_CLASS,
>> +.ids = pad_device_ids,
>> +.ops = {
>> +.add = xen_acpi_pad_add,
>
>.remove?
>
>[Jinsong]
Thanks! updated accordingly, w/ 2 comments left as below:
> +static const struct acpi_device_id pad_device_ids[] = {
> + {"ACPI000C", 0},
> + {"", 0},
> +};
> +
> +static struct acpi_driver xen_acpi_pad_driver = {
> + .name = "processor_aggregator",
> + .class =
Thanks! updated accordingly, w/ 2 comments left as below:
+static const struct acpi_device_id pad_device_ids[] = {
+ {ACPI000C, 0},
+ {, 0},
+};
+
+static struct acpi_driver xen_acpi_pad_driver = {
+ .name = processor_aggregator,
+ .class = ACPI_PROCESSOR_AGGREGATOR_CLASS,
Liu, Jinsong jinsong@intel.com 10/26/12 8:18 AM
+static struct acpi_driver xen_acpi_pad_driver = {
+.name = processor_aggregator,
+.class = ACPI_PROCESSOR_AGGREGATOR_CLASS,
+.ids = pad_device_ids,
+.ops = {
+.add = xen_acpi_pad_add,
.remove?
[Jinsong] .remove
] [PATCH 1/2] Xen acpi pad implement
Liu, Jinsong jinsong@intel.com 10/26/12 8:18 AM
+static struct acpi_driver xen_acpi_pad_driver = {
+.name = processor_aggregator,
+.class = ACPI_PROCESSOR_AGGREGATOR_CLASS,
+.ids = pad_device_ids,
+.ops = {
+.add
On 25.10.12 at 14:19, Liu, Jinsong jinsong@intel.com wrote:
--- /dev/null
+++ b/drivers/xen/xen_acpi_pad.c
@@ -0,0 +1,173 @@
+/*
+ * xen_acpi_pad.c - Xen pad interface
+ *
+ * Copyright (c) 2012, Intel Corporation.
+ *Author: Liu, Jinsong jinsong@intel.com
+ *
+ * This
Updated per Jan's comments.
Thanks,
Jinsong
===
From e6d4eceba9538c6873f9c970425e65e257fd9bf0 Mon Sep 17 00:00:00 2001
From: Liu, Jinsong jinsong@intel.com
Date: Sat, 27 Oct 2012 05:34:50 +0800
Subject: [PATCH 1/2] Xen acpi pad implement
PAD is acpi Processor Aggregator Device
On Fri, Oct 26, 2012 at 02:00:36PM +, Liu, Jinsong wrote:
Updated per Jan's comments.
Thanks,
Jinsong
===
From e6d4eceba9538c6873f9c970425e65e257fd9bf0 Mon Sep 17 00:00:00 2001
From: Liu, Jinsong jinsong@intel.com
Date: Sat, 27 Oct 2012 05:34:50 +0800
Subject:
54 matches
Mail list logo