From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH v5 0/5] Xen/ACPI: support sleep state entering on hardware reduced systems Date: Sat, 27 Jul 2013 16:05:08 +0200 Message-ID: <3763656.ERXJ8VWMlL__44506.6545777538$1374933434$gmane$org@vostro.rjw.lan> References: <1372427184-6197-1-git-send-email-benjamin.guthro@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1372427184-6197-1-git-send-email-benjamin.guthro@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ben Guthro Cc: Shane Wang , Bob Moore , linux-kernel@vger.kernel.org, xen-devel@lists.xen.org, linux-acpi@vger.kernel.org, tboot-devel@lists.sourceforge.net, Jan Beulich , Richard L Maliszewski , Gang Wei List-Id: xen-devel@lists.xenproject.org On Friday, June 28, 2013 09:46:19 AM Ben Guthro wrote: > In version 3.4 acpi_os_prepare_sleep() got introduced in parallel with > reduced hardware sleep support, and the two changes didn't get > synchronized: The new code doesn't call the hook function (if so > requested). Fix this, requiring a boolean parameter to be added to the > hook function to distinguish "extended" from "legacy" sleep. > > This requires adjusting TXT, but the adjustments only go as far as > failing the extended mode call (since, looking at the TXT interface, > there doesn't even appear to be precautions to deal with that > alternative interface). > > The hypervisor change underlying this is commit 62d1a69 ("ACPI: support > v5 (reduced HW) sleep interface") on the master branch of > git://xenbits.xen.org/xen.git. > > Signed-off-by: Jan Beulich > Signed-off-by: Ben Guthro > Cc: Richard L Maliszewski > Cc: Gang Wei > Cc: Shane Wang > Cc: Bob Moore > Cc: Rafael J. Wysocki > Cc: linux-acpi@vger.kernel.org > Cc: tboot-devel@lists.sourceforge.net > > v2: Extend description to include reference to hypervisor side change > v3: Split into multiple patches, separating subsystems > Remove bool parameters, in favor of u8 > v4: Remove linux/acpi.h dependencies > Further patch split to break out acpica from OSL > More bool vs u8 fixes > v5: Fix build of consumers of acpi_os_prepare_sleep() interface change, > so intermediate builds of partial patch series will not fail. > > Ben Guthro (5): > acpi: Remove need to include linux/acpi.h in common acpica code > acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path > acpi: Adjust linux acpi OS functions to new extended parameter > x86/tboot: Fail extended mode reduced hardware sleep > xen/acpi: notify xen when reduced hardware sleep is available I've just replied to [0/5] from the previous series, here: http://marc.info/?l=linux-acpi&m=137493312722239&w=2 Thanks, Rafael