RE: [PATCH 06/11] drivers: acpi: Add appropriate ifdef conditions in exdump.c

2013-12-17 Thread Zheng, Lv
Hi, I think this patch is useless. It is possible to include dump code into Linux kernel for debugging purposes. Thus we should do cleanup in different way for them. Thanks -Lv > From: Rashika Kheria [mailto:rashika.khe...@gmail.com] > Sent: Tuesday, December 17, 2013 5:24 PM > > Enclose

RE: [PATCH 05/11] drivers: acpi: Include appropriate header file in utstate.c

2013-12-17 Thread Zheng, Lv
are looking at, but in the master git > tree for ACPICA, the file accommon.h includes "acutils.h". > > > -Original Message- > > From: Rashika Kheria [mailto:rashika.khe...@gmail.com] > > Sent: Tuesday, December 17, 2013 1:22 AM > > To: linux-kernel@vger.kernel.org

RE: [PATCH 05/11] drivers: acpi: Include appropriate header file in utstate.c

2013-12-17 Thread Zheng, Lv
at, but in the master git tree for ACPICA, the file accommon.h includes acutils.h. -Original Message- From: Rashika Kheria [mailto:rashika.khe...@gmail.com] Sent: Tuesday, December 17, 2013 1:22 AM To: linux-kernel@vger.kernel.org Cc: Moore, Robert; Zheng, Lv; Wysocki, Rafael J; Len Brown; linux

RE: [PATCH 06/11] drivers: acpi: Add appropriate ifdef conditions in exdump.c

2013-12-17 Thread Zheng, Lv
Hi, I think this patch is useless. It is possible to include dump code into Linux kernel for debugging purposes. Thus we should do cleanup in different way for them. Thanks -Lv From: Rashika Kheria [mailto:rashika.khe...@gmail.com] Sent: Tuesday, December 17, 2013 5:24 PM Enclose functions

RE: [PATCH v2] SFI: fix compilation warnings

2013-12-10 Thread Zheng, Lv
Hi, > From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Andy Shevchenko > Sent: Tuesday, December 10, 2013 8:08 PM > > On Tue, 2013-12-10 at 01:30 +, Zheng, Lv wrote: > > Hi, Andy > > > > IMO: >

RE: [PATCH v2] SFI: fix compilation warnings

2013-12-10 Thread Zheng, Lv
Hi, From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Andy Shevchenko Sent: Tuesday, December 10, 2013 8:08 PM On Tue, 2013-12-10 at 01:30 +, Zheng, Lv wrote: Hi, Andy IMO: 1. sfi_acpi.h should include both linux/acpi.h and linux

RE: [PATCH v2] SFI: fix compilation warnings

2013-12-09 Thread Zheng, Lv
> -Original Message- > From: Andy Shevchenko [mailto:andriy.shevche...@linux.intel.com] > Sent: Monday, December 09, 2013 5:09 PM > To: Wysocki, Rafael J; linux-acpi @ vger . kernel . org; LKML; Zheng, Lv > Cc: Andy Shevchenko > Subject: [PATCH v2] SFI: fix compilation warnings >

RE: [PATCH v2] SFI: fix compilation warnings

2013-12-09 Thread Zheng, Lv
/lists/linux-acpi/msg47510.html Thanks and best regards -Lv -Original Message- From: Andy Shevchenko [mailto:andriy.shevche...@linux.intel.com] Sent: Monday, December 09, 2013 5:09 PM To: Wysocki, Rafael J; linux-acpi @ vger . kernel . org; LKML; Zheng, Lv Cc: Andy Shevchenko Subject

RE: [PATCH v3 6/6] ACPI/IBFT: Fix wrong inclusion in iSCSI boot firmware module.

2013-12-06 Thread Zheng, Lv
> From: Zheng, Lv > Sent: Friday, December 06, 2013 4:52 PM > > In Linux kernel, ACPICA is wrapped and safely exported by CONFIG_ACPI. So > all external modules should depend on CONFIG_ACPI rather than using ACPICA > header directly for stubbing. But if we moves inclusi

RE: [PATCH v3 5/6] ACPI/SFI: Fix wrong inclusion in SFI/ACPI wrapper - acpi_disabled linkage.

2013-12-06 Thread Zheng, Lv
Hi, > From: Zheng, Lv > Sent: Friday, December 06, 2013 4:52 PM > To: Wysocki, Rafael J; Brown, Len > > In Linux kernel, ACPICA is wrapped and safely exported by CONFIG_ACPI. So > all external modules should depend on CONFIG_ACPI rather than using ACPICA > header

RE: [PATCH v3 5/6] ACPI/SFI: Fix wrong acpi/acpi.h inclusion in SFI/ACPI wrapper - acpi_disabled linkage.

2013-12-06 Thread Zheng, Lv
Hi, From: Zheng, Lv Sent: Friday, December 06, 2013 4:52 PM To: Wysocki, Rafael J; Brown, Len In Linux kernel, ACPICA is wrapped and safely exported by CONFIG_ACPI. So all external modules should depend on CONFIG_ACPI rather than using ACPICA header directly for stubbing. But if we

RE: [PATCH v3 6/6] ACPI/IBFT: Fix wrong acpi/acpi.h inclusion in iSCSI boot firmware module.

2013-12-06 Thread Zheng, Lv
From: Zheng, Lv Sent: Friday, December 06, 2013 4:52 PM In Linux kernel, ACPICA is wrapped and safely exported by CONFIG_ACPI. So all external modules should depend on CONFIG_ACPI rather than using ACPICA header directly for stubbing. But if we moves acpi/acpi.h inclusions into #ifdef

RE: [PATCH] SFI: fix compilation warnings

2013-12-05 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki > Sent: Friday, December 06, 2013 10:01 AM > To: Zheng, Lv > Cc: Andy Shevchenko; Wysocki, Rafael J; linux-a...@vger.kernel.org; LKML > Subject: Re: [PATCH] SF

RE: [PATCH] SFI: fix compilation warnings

2013-12-05 Thread Zheng, Lv
> From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > Sent: Friday, December 06, 2013 9:26 AM > > On Friday, December 06, 2013 01:08:45 AM Zheng, Lv wrote: > > > From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > > > Sent: Friday, December 06, 2013 6:37 AM >

RE: [PATCH] SFI: fix compilation warnings

2013-12-05 Thread Zheng, Lv
> From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > Sent: Friday, December 06, 2013 6:37 AM > > On Thursday, December 05, 2013 07:26:37 PM Andy Shevchenko wrote: > > When build kernel with make W=1 we get the following compiler error. > > > > In file included from drivers/sfi/sfi_acpi.c:66:0:

RE: [PATCH] SFI: fix compilation warnings

2013-12-05 Thread Zheng, Lv
From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] Sent: Friday, December 06, 2013 6:37 AM On Thursday, December 05, 2013 07:26:37 PM Andy Shevchenko wrote: When build kernel with make W=1 we get the following compiler error. In file included from drivers/sfi/sfi_acpi.c:66:0:

RE: [PATCH] SFI: fix compilation warnings

2013-12-05 Thread Zheng, Lv
From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] Sent: Friday, December 06, 2013 9:26 AM On Friday, December 06, 2013 01:08:45 AM Zheng, Lv wrote: From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] Sent: Friday, December 06, 2013 6:37 AM On Thursday, December 05, 2013 07:26:37

RE: [PATCH] SFI: fix compilation warnings

2013-12-05 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki Sent: Friday, December 06, 2013 10:01 AM To: Zheng, Lv Cc: Andy Shevchenko; Wysocki, Rafael J; linux-a...@vger.kernel.org; LKML Subject: Re: [PATCH] SFI: fix compilation warnings

RE: [RFC part1 PATCH 5/7] ARM64 / ACPI: Introduce arm_core.c and its related head file

2013-12-03 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Hanjun Guo > Sent: Wednesday, December 04, 2013 12:37 AM > > introduce arm_core.c and its related head file, after this patch, > we can get ACPI tables from BIOS on ARM64 now. > > Signed-off-by: Al

RE: [RFC part1 PATCH 5/7] ARM64 / ACPI: Introduce arm_core.c and its related head file

2013-12-03 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Hanjun Guo Sent: Wednesday, December 04, 2013 12:37 AM introduce arm_core.c and its related head file, after this patch, we can get ACPI tables from BIOS on ARM64 now. Signed-off-by: Al Stone

RE: [PATCH v2] ACPI: Cleanup , and inclusions.

2013-11-27 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki > Sent: Wednesday, November 27, 2013 8:29 AM > > On Tuesday, November 26, 2013 03:29:05 PM Konrad Rzeszutek Wilk wrote: > > On Tue, Nov 26, 2013 at 09:29:33PM +0100, Rafael J.

RE: [PATCH v2] ACPI: Cleanup acpi/acpi.h, acpi/acpi_bus.h and acpi/acpi_drivers.h inclusions.

2013-11-27 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki Sent: Wednesday, November 27, 2013 8:29 AM On Tuesday, November 26, 2013 03:29:05 PM Konrad Rzeszutek Wilk wrote: On Tue, Nov 26, 2013 at 09:29:33PM +0100, Rafael J. Wysocki

RE: [UPDATE PATCH 4/4] ACPI: Add support to force header inclusion rules for .

2013-11-25 Thread Zheng, Lv
Hi, Rafael > From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > Sent: Tuesday, November 26, 2013 8:10 AM > To: Lv Zheng > Cc: Wysocki, Rafael J; Brown, Len; Zheng, Lv; linux-kernel@vger.kernel.org; > linux-a...@vger.kernel.org > Subject: Re: [UPDATE PATCH 4/4] ACPI: Ad

RE: [PATCH 2/4] ACPI: Cleanup and inclusions.

2013-11-25 Thread Zheng, Lv
Hi, Rafael Thanks for commenting. > From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > Sent: Tuesday, November 26, 2013 8:06 AM > > On Saturday, November 23, 2013 07:29:08 AM Lv Zheng wrote: > > From: Lv Zheng > > > > This patch enfoces inclusion instead of direct > > and inclusions. >

RE: [PATCH 2/4] ACPI: Cleanup acpi/acpi_bus.h and acpi/acpi_drivers.h inclusions.

2013-11-25 Thread Zheng, Lv
Hi, Rafael Thanks for commenting. From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] Sent: Tuesday, November 26, 2013 8:06 AM On Saturday, November 23, 2013 07:29:08 AM Lv Zheng wrote: From: Lv Zheng lv.zh...@intel.com This patch enfoces linux/acpi.h inclusion instead of direct

RE: [UPDATE PATCH 4/4] ACPI: Add support to force header inclusion rules for linux/acpi.h.

2013-11-25 Thread Zheng, Lv
Hi, Rafael From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] Sent: Tuesday, November 26, 2013 8:10 AM To: Lv Zheng Cc: Wysocki, Rafael J; Brown, Len; Zheng, Lv; linux-kernel@vger.kernel.org; linux-a...@vger.kernel.org Subject: Re: [UPDATE PATCH 4/4] ACPI: Add support to force header

RE: [PATCH 0/4] ACPICA: Stable material of ACPI executer fixes for linux-3.8.

2013-11-24 Thread Zheng, Lv
> From: Greg Kroah-Hartman [mailto:gre...@linuxfoundation.org] > Sent: Sunday, November 24, 2013 11:22 AM > > On Fri, Nov 01, 2013 at 02:58:16AM +0000, Zheng, Lv wrote: > > > From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > > > Sent: Thursday, October 31, 2013 8

RE: [PATCH 0/4] ACPICA: Stable material of ACPI executer fixes for linux-3.8.

2013-11-24 Thread Zheng, Lv
From: Greg Kroah-Hartman [mailto:gre...@linuxfoundation.org] Sent: Sunday, November 24, 2013 11:22 AM On Fri, Nov 01, 2013 at 02:58:16AM +, Zheng, Lv wrote: From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] Sent: Thursday, October 31, 2013 8:22 PM On Thursday, October 31

RE: [PATCH 0/4] ACPICA: Stable material of ACPI executer fixes for linux-3.8.

2013-10-31 Thread Zheng, Lv
> From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > Sent: Thursday, October 31, 2013 8:22 PM > > On Thursday, October 31, 2013 05:08:50 AM Greg Kroah-Hartman wrote: > > On Thu, Oct 31, 2013 at 12:39:21PM +0100, Rafael J. Wysocki wrote: > > > On Thursday, October 31, 2013 09:07:40 AM Lv Zheng

RE: [PATCH 0/4] ACPICA: Stable material of ACPI executer fixes for linux-3.8.

2013-10-31 Thread Zheng, Lv
From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] Sent: Thursday, October 31, 2013 8:22 PM On Thursday, October 31, 2013 05:08:50 AM Greg Kroah-Hartman wrote: On Thu, Oct 31, 2013 at 12:39:21PM +0100, Rafael J. Wysocki wrote: On Thursday, October 31, 2013 09:07:40 AM Lv Zheng wrote:

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-23 Thread Zheng, Lv
> From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] > Sent: Tuesday, September 17, 2013 4:37 PM > > On Mon, Sep 16, 2013 at 11:35:56PM +0000, Zheng, Lv wrote: > > > From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] > > > Sent: Mond

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-23 Thread Zheng, Lv
From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] Sent: Tuesday, September 17, 2013 4:37 PM On Mon, Sep 16, 2013 at 11:35:56PM +, Zheng, Lv wrote: From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] Sent: Monday, September 16, 2013 4:11 PM On Mon, Sep

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-16 Thread Zheng, Lv
> From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] > Sent: Monday, September 16, 2013 4:11 PM > > On Mon, Sep 16, 2013 at 01:21:53AM +0000, Zheng, Lv wrote: > > > A pseudo device may be created to access the GPIO operation region fields > > &g

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-16 Thread Zheng, Lv
From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] Sent: Monday, September 16, 2013 4:11 PM On Mon, Sep 16, 2013 at 01:21:53AM +, Zheng, Lv wrote: A pseudo device may be created to access the GPIO operation region fields provided by one GPIO device. The pseudo device

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-15 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Zheng, Lv > Sent: Monday, September 16, 2013 8:47 AM > > > From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] > > Sent: Sunday, September 15, 2013 2:52 PM >

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-15 Thread Zheng, Lv
> From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] > Sent: Sunday, September 15, 2013 2:52 PM > > On Sat, Sep 14, 2013 at 12:10:37AM +0000, Zheng, Lv wrote: > > Is it possible to install the handler for ACPI_ROOT_OBJECT? > > Can it be achieved by imple

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-15 Thread Zheng, Lv
From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] Sent: Sunday, September 15, 2013 2:52 PM On Sat, Sep 14, 2013 at 12:10:37AM +, Zheng, Lv wrote: Is it possible to install the handler for ACPI_ROOT_OBJECT? Can it be achieved by implementing a setup callback? Yes

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-15 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Zheng, Lv Sent: Monday, September 16, 2013 8:47 AM From: Mika Westerberg [mailto:mika.westerb...@linux.intel.com] Sent: Sunday, September 15, 2013 2:52 PM On Sat, Sep 14, 2013 at 12:10:37AM

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-13 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Mika Westerberg > Sent: Friday, September 13, 2013 11:15 PM > > GPIO operation regions is a new feature introduced in ACPI 5.0 > specification. In practise it means that now ASL code can toggle

RE: [PATCH 2/2] gpio / ACPI: add support for GPIO operation regions

2013-09-13 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Mika Westerberg Sent: Friday, September 13, 2013 11:15 PM GPIO operation regions is a new feature introduced in ACPI 5.0 specification. In practise it means that now ASL code can toggle GPIOs with

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-28 Thread Zheng, Lv
what we are talking about are the future, we can just live with the current approaches. We can find a way in the future when the conflicts do happen. Thanks and best regards -Lv > On Saturday, July 27, 2013 2:04 AM konrad wilk >> On 7/25/2013 10:51 PM, Zheng, Lv wrote: > >

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-28 Thread Zheng, Lv
> On Friday, July 26, 2013 10:49 PM Rafael J. Wysocki wrote: > > On Friday, July 26, 2013 01:54:00 AM Zheng, Lv wrote: > > > On Friday, July 26, 2013 5:29 AM Rafael J. Wysocki wrote: > > > On Tuesday, July 23, 2013 04:09:43 PM Lv Zheng wrote: > > > > Thi

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-28 Thread Zheng, Lv
> On Friday, July 26, 2013 10:01 PM Rafael J. Wysocki wrote: > > On Friday, July 26, 2013 12:47:44 AM Zheng, Lv wrote: > > > > > On Friday, July 26, 2013 4:27 AM Rafael J. Wysocki wrote: > > > > > > On Tuesday, July 23, 2013 04:09:43 PM Lv Zheng wrote:

RE: [PATCH 08/13] ACPI/IPMI: Cleanup several acpi_ipmi_device members

2013-07-28 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki > > On Friday, July 26, 2013 01:25:12 AM Zheng, Lv wrote: > > > From: Rafael J. Wysocki [mailto:r...@sisk.pl] > > > Sent: Friday, July 26, 2013 6:26 A

RE: [PATCH 08/13] ACPI/IPMI: Cleanup several acpi_ipmi_device members

2013-07-28 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki On Friday, July 26, 2013 01:25:12 AM Zheng, Lv wrote: From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Friday, July 26, 2013 6:26 AM On Tuesday, July 23, 2013 04:10

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-28 Thread Zheng, Lv
On Friday, July 26, 2013 10:01 PM Rafael J. Wysocki wrote: On Friday, July 26, 2013 12:47:44 AM Zheng, Lv wrote: On Friday, July 26, 2013 4:27 AM Rafael J. Wysocki wrote: On Tuesday, July 23, 2013 04:09:43 PM Lv Zheng wrote: This patch adds reference couting for ACPI operation

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-28 Thread Zheng, Lv
On Friday, July 26, 2013 10:49 PM Rafael J. Wysocki wrote: On Friday, July 26, 2013 01:54:00 AM Zheng, Lv wrote: On Friday, July 26, 2013 5:29 AM Rafael J. Wysocki wrote: On Tuesday, July 23, 2013 04:09:43 PM Lv Zheng wrote: This patch adds reference couting for ACPI operation region

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-28 Thread Zheng, Lv
what we are talking about are the future, we can just live with the current approaches. We can find a way in the future when the conflicts do happen. Thanks and best regards -Lv On Saturday, July 27, 2013 2:04 AM konrad wilk On 7/25/2013 10:51 PM, Zheng, Lv wrote: From: Konrad Rzeszutek Wilk

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-26 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Zheng, Lv > Sent: Friday, July 26, 2013 9:54 AM > To: Rafael J. Wysocki > Cc: Wysocki, Rafael J; Brown, Len; linux-kernel@vger.kernel.org; > linux-a...@vger.kernel.org > Subj

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-26 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Zheng, Lv > Sent: Friday, July 26, 2013 8:48 AM > > > > > From: Rafael J. Wysocki [mailto:r...@sisk.pl] > > Sent: Friday, July 26, 2013 4:27 AM > > > >

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-26 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Zheng, Lv Sent: Friday, July 26, 2013 8:48 AM From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Friday, July 26, 2013 4:27 AM On Tuesday, July 23, 2013 04:09:43 PM Lv Zheng wrote

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-26 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Zheng, Lv Sent: Friday, July 26, 2013 9:54 AM To: Rafael J. Wysocki Cc: Wysocki, Rafael J; Brown, Len; linux-kernel@vger.kernel.org; linux-a...@vger.kernel.org Subject: RE: [PATCH 06/13] ACPI/IPMI

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-25 Thread Zheng, Lv
> From: Konrad Rzeszutek Wilk [mailto:konrad.w...@oracle.com] > Sent: Thursday, July 25, 2013 8:04 PM > > CC-ing some of the tboot maintainers. > > As what I've said, it's up to the others to determine if the patch is OK. > > I just need to make my concerns visible in the community. :-) > > If I

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-25 Thread Zheng, Lv
> From: Rafael J. Wysocki [mailto:r...@sisk.pl] > Sent: Friday, July 26, 2013 5:29 AM > > On Tuesday, July 23, 2013 04:09:43 PM Lv Zheng wrote: > > This patch adds reference couting for ACPI operation region handlers to fix > > races caused by the ACPICA address space callback invocations. > > >

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
> From: Corey Minyard [mailto:tcminy...@gmail.com] > Sent: Friday, July 26, 2013 8:48 AM > > On 07/25/2013 07:16 PM, Zheng, Lv wrote: > >> > >> If I understand this correctly, the problem would be if: > >> > >> rem_time

RE: [PATCH 08/13] ACPI/IPMI: Cleanup several acpi_ipmi_device members

2013-07-25 Thread Zheng, Lv
> From: Rafael J. Wysocki [mailto:r...@sisk.pl] > Sent: Friday, July 26, 2013 6:26 AM > > On Tuesday, July 23, 2013 04:10:06 PM Lv Zheng wrote: > > This is a trivial patch: > > 1. Deletes a member of the acpi_ipmi_device - smi_data which is not > >actually used. > > 2. Updates a member of the

RE: [PATCH 07/13] ACPI/IPMI: Add reference counting for ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki > Sent: Friday, July 26, 2013 6:23 AM > > On Tuesday, July 23, 2013 04:09:54 PM Lv Zheng wrote: > > This patch adds reference counting for ACPI IPMI transfers to tune the > >

RE: [PATCH 04/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI user

2013-07-25 Thread Zheng, Lv
> From: Rafael J. Wysocki [mailto:r...@sisk.pl] > Sent: Friday, July 26, 2013 5:59 AM > > On Tuesday, July 23, 2013 04:09:26 PM Lv Zheng wrote: > > This patch uses reference counting to fix the race caused by the > > unprotected ACPI IPMI user. > > > > As the acpi_ipmi_device->user_interface

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-25 Thread Zheng, Lv
> From: Rafael J. Wysocki [mailto:r...@sisk.pl] > Sent: Friday, July 26, 2013 4:27 AM > > On Tuesday, July 23, 2013 04:09:43 PM Lv Zheng wrote: > > This patch adds reference couting for ACPI operation region handlers > > to fix races caused by the ACPICA address space callback invocations. > >

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
> > On Thursday, July 25, 2013 03:09:35 AM Zheng, Lv wrote: > > >> -stable according to the previous conversation. > > >> > > >>> From: Rafael J. Wysocki [mailto:r...@sisk.pl] > > >>> Sent: Thursday, July 25, 2013 7:38 AM > > >>

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
> From: Corey Minyard [mailto:tcminy...@gmail.com] > Sent: Friday, July 26, 2013 2:13 AM > > On 07/25/2013 07:06 AM, Rafael J. Wysocki wrote: > > On Thursday, July 25, 2013 03:09:35 AM Zheng, Lv wrote: > >> -stable according to the previous conversation. > &g

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
> From: Rafael J. Wysocki [mailto:r...@sisk.pl] > Sent: Thursday, July 25, 2013 8:07 PM > > On Thursday, July 25, 2013 03:09:35 AM Zheng, Lv wrote: > > -stable according to the previous conversation. > > > > > From: Rafael J. Wysocki [mailto:r...@sisk.pl] >

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Thursday, July 25, 2013 8:07 PM On Thursday, July 25, 2013 03:09:35 AM Zheng, Lv wrote: -stable according to the previous conversation. From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Thursday, July 25, 2013 7:38 AM

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
From: Corey Minyard [mailto:tcminy...@gmail.com] Sent: Friday, July 26, 2013 2:13 AM On 07/25/2013 07:06 AM, Rafael J. Wysocki wrote: On Thursday, July 25, 2013 03:09:35 AM Zheng, Lv wrote: -stable according to the previous conversation. From: Rafael J. Wysocki [mailto:r...@sisk.pl

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
:09:35 AM Zheng, Lv wrote: -stable according to the previous conversation. From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Thursday, July 25, 2013 7:38 AM On Tuesday, July 23, 2013 04:09:15 PM Lv Zheng wrote: This patch fixes races caused by unprotected ACPI IPMI transfers

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-25 Thread Zheng, Lv
From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Friday, July 26, 2013 4:27 AM On Tuesday, July 23, 2013 04:09:43 PM Lv Zheng wrote: This patch adds reference couting for ACPI operation region handlers to fix races caused by the ACPICA address space callback invocations. ACPICA

RE: [PATCH 04/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI user

2013-07-25 Thread Zheng, Lv
From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Friday, July 26, 2013 5:59 AM On Tuesday, July 23, 2013 04:09:26 PM Lv Zheng wrote: This patch uses reference counting to fix the race caused by the unprotected ACPI IPMI user. As the acpi_ipmi_device-user_interface check in

RE: [PATCH 07/13] ACPI/IPMI: Add reference counting for ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. Wysocki Sent: Friday, July 26, 2013 6:23 AM On Tuesday, July 23, 2013 04:09:54 PM Lv Zheng wrote: This patch adds reference counting for ACPI IPMI transfers to tune the locking

RE: [PATCH 08/13] ACPI/IPMI: Cleanup several acpi_ipmi_device members

2013-07-25 Thread Zheng, Lv
From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Friday, July 26, 2013 6:26 AM On Tuesday, July 23, 2013 04:10:06 PM Lv Zheng wrote: This is a trivial patch: 1. Deletes a member of the acpi_ipmi_device - smi_data which is not actually used. 2. Updates a member of the

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-25 Thread Zheng, Lv
From: Corey Minyard [mailto:tcminy...@gmail.com] Sent: Friday, July 26, 2013 8:48 AM On 07/25/2013 07:16 PM, Zheng, Lv wrote: If I understand this correctly, the problem would be if: rem_time = wait_for_completion_timeout(tx_msg-tx_complete

RE: [PATCH 06/13] ACPI/IPMI: Add reference counting for ACPI operation region handlers

2013-07-25 Thread Zheng, Lv
From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Friday, July 26, 2013 5:29 AM On Tuesday, July 23, 2013 04:09:43 PM Lv Zheng wrote: This patch adds reference couting for ACPI operation region handlers to fix races caused by the ACPICA address space callback invocations. ACPICA

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-25 Thread Zheng, Lv
From: Konrad Rzeszutek Wilk [mailto:konrad.w...@oracle.com] Sent: Thursday, July 25, 2013 8:04 PM CC-ing some of the tboot maintainers. As what I've said, it's up to the others to determine if the patch is OK. I just need to make my concerns visible in the community. :-) If I

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-24 Thread Zheng, Lv
-stable according to the previous conversation. > From: Rafael J. Wysocki [mailto:r...@sisk.pl] > Sent: Thursday, July 25, 2013 7:38 AM > > On Tuesday, July 23, 2013 04:09:15 PM Lv Zheng wrote: > > This patch fixes races caused by unprotected ACPI IPMI transfers. > > > > We can see the following

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-24 Thread Zheng, Lv
. On Wed, Jul 24, 2013 at 9:28 PM, Zheng, Lv wrote: Let me just give an example to let you know the difficulties for ACPICA developers to merge Xen's acpi_os_prepare_sleep. The original logic in the acpi_hw_legacy_sleep is: 111         /* Get current value of PM1A control */ 112 113         status

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-24 Thread Zheng, Lv
day, July 25, 2013 12:32 AM > To: Ben Guthro > Cc: Moore, Robert; Zheng, Lv; Jan Beulich; Rafael J . Wysocki; > linux-kernel@vger.kernel.org; linux-a...@vger.kernel.org; > xen-de...@lists.xen.org > Subject: Re: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced > har

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-24 Thread Zheng, Lv
s change and why it is > being considered? > > Thanks, > Bob > > > > -Original Message- > > From: Ben Guthro [mailto:benjamin.gut...@citrix.com] > > Sent: Wednesday, July 24, 2013 6:23 AM > > To: Moore, Robert > > Cc: Zheng, Lv; Konrad

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-24 Thread Zheng, Lv
Hi, Sorry for the delayed response. > From: Ben Guthro [mailto:benjamin.gut...@citrix.com] > Sent: Tuesday, July 02, 2013 7:43 PM > > > On 07/02/2013 02:19 AM, Zheng, Lv wrote: > > Thanks for your efforts! > > > > I wonder if it is possible to r

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-24 Thread Zheng, Lv
, Bob -Original Message- From: Ben Guthro [mailto:benjamin.gut...@citrix.com] Sent: Wednesday, July 24, 2013 6:23 AM To: Moore, Robert Cc: Zheng, Lv; Konrad Rzeszutek Wilk; Jan Beulich; Rafael J . Wysocki; linux-kernel@vger.kernel.org; linux-a...@vger.kernel.org; xen- de

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-24 Thread Zheng, Lv
(). Hope the above example can make my concern clearer now. :-) Thanks -Lv -Original Message- From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Konrad Rzeszutek Wilk Sent: Thursday, July 25, 2013 12:32 AM To: Ben Guthro Cc: Moore, Robert; Zheng

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-24 Thread Zheng, Lv
. On Wed, Jul 24, 2013 at 9:28 PM, Zheng, Lv lv.zh...@intel.com wrote: Let me just give an example to let you know the difficulties for ACPICA developers to merge Xen's acpi_os_prepare_sleep. The original logic in the acpi_hw_legacy_sleep is: 111         /* Get current value of PM1A control */ 112 113

RE: [PATCH 03/13] ACPI/IPMI: Fix race caused by the unprotected ACPI IPMI transfers

2013-07-24 Thread Zheng, Lv
-stable according to the previous conversation. From: Rafael J. Wysocki [mailto:r...@sisk.pl] Sent: Thursday, July 25, 2013 7:38 AM On Tuesday, July 23, 2013 04:09:15 PM Lv Zheng wrote: This patch fixes races caused by unprotected ACPI IPMI transfers. We can see the following crashes

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-24 Thread Zheng, Lv
Hi, Sorry for the delayed response. From: Ben Guthro [mailto:benjamin.gut...@citrix.com] Sent: Tuesday, July 02, 2013 7:43 PM On 07/02/2013 02:19 AM, Zheng, Lv wrote: Thanks for your efforts! I wonder if it is possible to remove the argument - u8 extended and convert

RE: [PATCH 01/13] ACPI/IPMI: Fix potential response buffer overflow

2013-07-23 Thread Zheng, Lv
> From: Zheng, Lv > Sent: Wednesday, July 24, 2013 8:22 AM > > > From: linux-acpi-ow...@vger.kernel.org > > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Greg KH > > Sent: Tuesday, July 23, 2013 10:54 PM > > > > On Tue, Jul 23, 2013 at 04:08:59PM

RE: [PATCH 01/13] ACPI/IPMI: Fix potential response buffer overflow

2013-07-23 Thread Zheng, Lv
> From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Greg KH > Sent: Tuesday, July 23, 2013 10:54 PM > > On Tue, Jul 23, 2013 at 04:08:59PM +0800, Lv Zheng wrote: > > This patch enhances sanity checks on message size to avoid potential > > buffer

RE: [PATCH 01/13] ACPI/IPMI: Fix potential response buffer overflow

2013-07-23 Thread Zheng, Lv
From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Greg KH Sent: Tuesday, July 23, 2013 10:54 PM On Tue, Jul 23, 2013 at 04:08:59PM +0800, Lv Zheng wrote: This patch enhances sanity checks on message size to avoid potential buffer overflow.

RE: [PATCH 01/13] ACPI/IPMI: Fix potential response buffer overflow

2013-07-23 Thread Zheng, Lv
From: Zheng, Lv Sent: Wednesday, July 24, 2013 8:22 AM From: linux-acpi-ow...@vger.kernel.org [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Greg KH Sent: Tuesday, July 23, 2013 10:54 PM On Tue, Jul 23, 2013 at 04:08:59PM +0800, Lv Zheng wrote: This patch enhances sanity

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-02 Thread Zheng, Lv
Thanks for your efforts! I wonder if it is possible to remove the argument - "u8 extended" and convert "pm1a_control, pm1b_control" into some u8 values that are equivalent to "acpi_gbl_sleep_type_a, acpi_gbl_sleep_type_b" in the legacy sleep path. It can also simplify Xen codes. As in ACPI

RE: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path

2013-07-02 Thread Zheng, Lv
Thanks for your efforts! I wonder if it is possible to remove the argument - u8 extended and convert pm1a_control, pm1b_control into some u8 values that are equivalent to acpi_gbl_sleep_type_a, acpi_gbl_sleep_type_b in the legacy sleep path. It can also simplify Xen codes. As in ACPI

RE: [PATCH] ACPI: remove "config ACPI_DEBUG_FUNC_TRACE"

2013-04-07 Thread Zheng, Lv
> > > > Kconfig symbol ACPI_DEBUG_FUNC_TRACE was only used (through its > > > > corresponding macro) in drivers/acpi/acpica/acmacros.h. That macro > > > > was removed from that header in v3.8, with commit > > > > 86ff0e508f88eda6e479a897476026055831d2d8 ("ACPICA: Fix unmerged > > > > acmacros.h

RE: [PATCH] ACPI: remove "config ACPI_DEBUG_FUNC_TRACE"

2013-04-07 Thread Zheng, Lv
> > > Kconfig symbol ACPI_DEBUG_FUNC_TRACE was only used (through its > > > corresponding macro) in drivers/acpi/acpica/acmacros.h. That macro > > > was removed from that header in v3.8, with commit > > > 86ff0e508f88eda6e479a897476026055831d2d8 ("ACPICA: Fix unmerged > > > acmacros.h

RE: [PATCH] ACPI: remove config ACPI_DEBUG_FUNC_TRACE

2013-04-07 Thread Zheng, Lv
Kconfig symbol ACPI_DEBUG_FUNC_TRACE was only used (through its corresponding macro) in drivers/acpi/acpica/acmacros.h. That macro was removed from that header in v3.8, with commit 86ff0e508f88eda6e479a897476026055831d2d8 (ACPICA: Fix unmerged acmacros.h divergences.). That commit

RE: [PATCH] ACPI: remove config ACPI_DEBUG_FUNC_TRACE

2013-04-07 Thread Zheng, Lv
Kconfig symbol ACPI_DEBUG_FUNC_TRACE was only used (through its corresponding macro) in drivers/acpi/acpica/acmacros.h. That macro was removed from that header in v3.8, with commit 86ff0e508f88eda6e479a897476026055831d2d8 (ACPICA: Fix unmerged acmacros.h divergences.). That

RE: [PATCH] ACPI: remove "config ACPI_DEBUG_FUNC_TRACE"

2013-04-06 Thread Zheng, Lv
> > Kconfig symbol ACPI_DEBUG_FUNC_TRACE was only used (through its > > corresponding macro) in drivers/acpi/acpica/acmacros.h. That macro was > > removed from that header in v3.8, with commit > > 86ff0e508f88eda6e479a897476026055831d2d8 ("ACPICA: Fix unmerged > > acmacros.h divergences."). That

RE: [PATCH] ACPI: remove config ACPI_DEBUG_FUNC_TRACE

2013-04-06 Thread Zheng, Lv
Kconfig symbol ACPI_DEBUG_FUNC_TRACE was only used (through its corresponding macro) in drivers/acpi/acpica/acmacros.h. That macro was removed from that header in v3.8, with commit 86ff0e508f88eda6e479a897476026055831d2d8 (ACPICA: Fix unmerged acmacros.h divergences.). That commit did

RE: [RESEND PATCH 3/6] acpi: Remove the leading spaces of "finish_override" label in acpi_tb_table_override().

2013-03-08 Thread Zheng, Lv
> > > https://github.com/acpica/acpica/tree/master/generate/linux > > Hi Lv, > > > > Thanks for telling me this. :) > > > > One more thing, if I want to fix something in acpica, such as this > > patch set, who and which mail list should I send patches to ? > > Please post them to

RE: [RESEND PATCH 3/6] acpi: Remove the leading spaces of "finish_override" label in acpi_tb_table_override().

2013-03-08 Thread Zheng, Lv
/generate/linux Thanks -Lv > -Original Message- > From: Tang Chen [mailto:tangc...@cn.fujitsu.com] > Sent: Thursday, March 07, 2013 6:38 PM > To: l...@kernel.org; r...@sisk.pl; Moore, Robert; Zheng, Lv; > ming.m@intel.com; m...@selenic.com; herb...@gondor.apana.org.au; > r..

RE: [RESEND PATCH 3/6] acpi: Remove the leading spaces of finish_override label in acpi_tb_table_override().

2013-03-08 Thread Zheng, Lv
/generate/linux Thanks -Lv -Original Message- From: Tang Chen [mailto:tangc...@cn.fujitsu.com] Sent: Thursday, March 07, 2013 6:38 PM To: l...@kernel.org; r...@sisk.pl; Moore, Robert; Zheng, Lv; ming.m@intel.com; m...@selenic.com; herb...@gondor.apana.org.au; r...@landley.net Cc: linux

RE: [RESEND PATCH 3/6] acpi: Remove the leading spaces of finish_override label in acpi_tb_table_override().

2013-03-08 Thread Zheng, Lv
https://github.com/acpica/acpica/tree/master/generate/linux Hi Lv, Thanks for telling me this. :) One more thing, if I want to fix something in acpica, such as this patch set, who and which mail list should I send patches to ? Please post them to linux-a...@vger.kernel.org with

RE: linux-next: manual merge of the acpi tree with the pm tree

2013-02-16 Thread Zheng, Lv
> Hi Len, > > On Mon, 11 Feb 2013 18:34:06 -0500 Len Brown wrote: > > > > BTW. Rafael's "pm" tree now carries the ACPI patch stream, so it is > > probably a mis-representation to call my tree the "acpi" tree. > > My tree is primarily focused on the "idle" part of pm these days. > > OK, I have

RE: linux-next: manual merge of the acpi tree with the pm tree

2013-02-16 Thread Zheng, Lv
Hi Len, On Mon, 11 Feb 2013 18:34:06 -0500 Len Brown l...@kernel.org wrote: BTW. Rafael's pm tree now carries the ACPI patch stream, so it is probably a mis-representation to call my tree the acpi tree. My tree is primarily focused on the idle part of pm these days. OK, I have

<    4   5   6   7   8   9   10   >