On Monday, July 22, 2013 08:44:08 AM Ben Guthro wrote: > On Mon, Jul 8, 2013 at 9:08 AM, Ben Guthro <benjamin.gut...@citrix.com> wrote: > > > > > > On 07/08/2013 09:10 AM, Rafael J. Wysocki wrote: > >> On Sunday, July 07, 2013 08:13:15 PM Ben Guthro wrote: > >>> On Mon, Jul 1, 2013 at 7:48 AM, Ben Guthro <benjamin.gut...@citrix.com> > >>> 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 <jbeul...@suse.com> > >>>> Signed-off-by: Ben Guthro <benjamin.gut...@citrix.com> > >>>> Cc: Richard L Maliszewski <richard.l.maliszew...@intel.com> > >>>> Cc: Gang Wei <gang....@intel.com> > >>>> Cc: Shane Wang <shane.w...@intel.com> > >>>> Cc: Bob Moore <robert.mo...@intel.com> > >>>> Cc: Rafael J. Wysocki <r...@sisk.pl> > >>>> Cc: linux-a...@vger.kernel.org > >>>> Cc: tboot-de...@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. > >>>> v6: Rebased to linux-pm linux-next branch > >>>> Added warning in tboot early return code > >>>> Added Reviewed-by lines > >>>> > >>>> 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/xen/tboot: 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 > >>> > >>> Rafael, Bob - > >>> > >>> Is this version of the series something that you may consider taking? > >>> If not - is there something specific you would like to see addressed? > >> > >> Well, I'm afraid you need to give us some more time to process that, > >> sorry. > > > > My apologies - I know it is a busy time during the merge window. I did > > not mean to seem impatient. > > > > With the volume of things on the mailing list, I merely wanted to try to > > avoid the series getting buried. If a week is too frequent to re-bump a > > series, do you have a set of recommendations of how to best interact > > with your workflow? > > Rafael (et al.) > > I never heard back on this point of how you prefer your workflow to > proceed - that is, if I haven't heard anything in a couple weeks - > should I try to re-bump the thread? > > Since this was sent around the time of the merge window, I realize > there were other things to occupy people's time. > Now that rc2 is out - I'm hoping that the dust has settled a bit, such > that there may be a cycle, or two to review this series.
I've just replied to a [0/5] from a previous series, here: http://marc.info/?l=linux-acpi&m=137493312722239&w=2 Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/