linux-kernel - Re: [PATCH v2 0/2] ACPI: EC: Install EC address space handler at the namespace root

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Thu, 16 May 2024 12:21:27 +0200
From: "Rafael J. Wysocki" <rafael@...nel.org>
To: Hans de Goede <hdegoede@...hat.com>
Cc: "Rafael J. Wysocki" <rafael@...nel.org>, "Rafael J. Wysocki" <rjw@...ysocki.net>, 
	Linux ACPI <linux-acpi@...r.kernel.org>, LKML <linux-kernel@...r.kernel.org>, 
	Andy Shevchenko <andriy.shevchenko@...ux.intel.com>, 
	Mario Limonciello <mario.limonciello@....com>, Armin Wolf <w_armin@....de>, 
	Heikki Krogerus <heikki.krogerus@...ux.intel.com>
Subject: Re: [PATCH v2 0/2] ACPI: EC: Install EC address space handler at the
 namespace root

Hi,

On Thu, May 16, 2024 at 12:14 PM Hans de Goede <hdegoede@...hat.com> wrote:
>
> Hi,
>
> On 5/16/24 12:09 PM, Rafael J. Wysocki wrote:
> > Hi,
> >
> > On Thu, May 16, 2024 at 11:50 AM Hans de Goede <hdegoede@...hatcom> wrote:
> >>
> >> Hi,
> >>
> >> On 5/16/24 10:37 AM, Rafael J. Wysocki wrote:
> >>> On Thu, May 16, 2024 at 10:35 AM Hans de Goede <hdegoede@...hat.com> wrote:
> >>>>
> >>>> Hi,
> >>>>
> >>>> On 5/15/24 9:39 PM, Rafael J. Wysocki wrote:
> >>>>> Hi Everyone,
> >>>>>
> >>>>> This is an update of
> >>>>>
> >>>>> https://lore.kernel.org/linux-acpi/5787281.DvuYhMxLoT@kreacher/
> >>>>>
> >>>>> which was a follow up for the discussion in:
> >>>>>
> >>>>> https://lore.kernel.org/linux-acpi/CAJZ5v0hiXdv08PRcop7oSYqgr_g5rwzRTj7HgdNCCGjXeV44zA@mail.gmail.com/T/#t
> >>>>>
> >>>>> Patch [1/2] has been updated to avoid possible issues related to
> >>>>> systems with defective platform firmware and patch [2/2] is a resend
> >>>>> with a couple of tags added.
> >>>>
> >>>> Thanks, the series looks good to me:
> >>>>
> >>>> Reviewed-by: Hans de Goede <hdegoede@...hat.com>
> >>>>
> >>>> for the series.
> >>>>
> >>>> I assume you are going to send this in as a fix for 6.10 ?
> >>>
> >>> Yes, I am.
> >>>
> >>>> In that case feel free to merge both patches through the
> >>>> linux-pm tree.
> >>>
> >>> Thank you!
> >>
> >> Hmm, I just realized that this:
> >>
> >> https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/commit/?h=for-next&id=c663b26972eae7d2a614f584c92a266fe9a2d44c
> >>
> >> Is part of the main pdx86 pull-request for 6.10 which I'm going to
> >> send to Linus in the next 10 minutes or so. So that is going to
> >> conflict with your 2/2.
> >>
> >> Options:
> >>
> >> a) You only send 1/2 upstream as a fix and I'll then send a rebased
> >> 2/2 upstream as part of the first pdx86 pull-request.
> >>
> >> b) You merge the git://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git
> >> platform-drivers-x86-v6.10-1 tag (which is the tag for the pull-request
> >> I'm about to send to Linus) and rebase on top of that before sending
> >> a pull-request for both to Linus.
> >
> > I would rather wait for Linus to merge your PR and merge my changes on
> > top of his merge.
>
> That is fine too. I just send out the pull-request so hopefully Linus will
> merge it soon(ish).
>
> Note (stating the obvious) when rebasing 2/2 you will pretty much need to
> remove all the new code added by:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/commit/?id=c663b26972eae7d2a614f584c92a266fe9a2d44c

I see, thanks for the notice!