Re: [PATCH v3 2/2] ACPI,acpi_memhotplug: Remove acpi_memory_info->failedbit

[Yasuaki Ishimatsu]

Hi Rafael,

2013/04/02 21:57, Rafael J. Wysocki wrote:
> On Tuesday, April 02, 2013 09:12:44 AM Yasuaki Ishimatsu wrote:
> > Hi Rafael,
> >
> > 2013/04/02 8:48, Rafael J. Wysocki wrote:
> > > On Thursday, March 28, 2013 08:58:03 AM Yasuaki Ishimatsu wrote:
> > > > Hi Rafael,
> > > >
> > > > 2013/03/26 22:41, Rafael J. Wysocki wrote:
> > > > > On Tuesday, March 26, 2013 08:57:05 AM Yasuaki Ishimatsu wrote:
> > > > > > 2013/03/23 5:24, Toshi Kani wrote:
> > > > > > > On Fri, 2013-03-22 at 10:53 +0900, Yasuaki Ishimatsu wrote:
> > > > > > > > acpi_memory_info has enabled bit and failed bit for controlling memory
> > > > > > > > hotplug. But we don't need to keep both bits.
> > > > > > > >
> > > > > > > > The patch removes acpi_memory_info->failed bit.
> > > > > > > >
> > > > > > > > Signed-off-by: yasuaki ishimatsu <isimatu.yasuaki@xxxxxxxxxxxxxx>
> > > > > > >
> > > > > > > Thanks for the update.  It looks good.  For the series:
> > > > > > >
> > > > > > > Acked-by: Toshi Kani <toshi.kani@xxxxxx>
> > > > > >
> > > > > > Thanks!
> > > > >
> > > > > Both patches applied to linux-pm.git/linux-next.
> > > >
> > > > Thank you for applying both patches.
> > > > If you don't mind, I hope these patches are merged into linux-3.9
> > > > since this is bug fix.
> > >
> > > I honestly don't think that [2/2] is 3.9 material, but I can push [1/2]
> > > for 3.9 provided that you also regard it as material for 3.8-stable.
> >
> > linux-3.8 does not support physical memory hot remove. It has been supported
> > since linux-3.9. So I don't need [1/2] patch to linux-3.8 but need it to
> > linux-3.9. So please push [1/2] patch to linux-3.9.
>
> Well, I don't think it would be a good idea to push that patch for 3.9 so late
> in the cycle.  Yes, it would make memory hotplug work in 3.9, but that might
> surprise some people and cause problems to happen as a result and we wouldn't
> have the time to address them before the 3.9 release.
>
> If that's not a big problem, I'd prefer to defer that patch until the 3.10
> merge window is open (which is just about 2 weeks from now) and if need be we
> can still push it for 3.9-stable afterward.

Sorry for bothering you by my mention.
I understood your thought. Please merge this patch to 3.10 and push if for
3.9-stable.

Thanks,
Yasuaki Ishimatsu

> Thanks,
> Rafael


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/