linux-kernel - Re: [PATCH v9 12/20] acpi/generic_event_device: add logic to detect if HEST addr is available

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] [day] [month] [year] [list]

Message-ID: <20250602132436.1e0e83aa@foz.lan>
Date: Mon, 2 Jun 2025 13:28:08 +0200
From: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
To: Igor Mammedov <imammedo@...hat.com>
Cc: "Michael S. Tsirkin" <mst@...hat.com>, Jonathan Cameron
 <Jonathan.Cameron@...wei.com>, Shiju Jose <shiju.jose@...wei.com>,
 qemu-arm@...gnu.org, qemu-devel@...gnu.org, Philippe Mathieu-Daudé <philmd@...aro.org>, Ani Sinha
 <anisinha@...hat.com>, Eduardo Habkost <eduardo@...kost.net>, Marcel
 Apfelbaum <marcel.apfelbaum@...il.com>, Peter Maydell
 <peter.maydell@...aro.org>, Shannon Zhao <shannon.zhaosl@...il.com>, Yanan
 Wang <wangyanan55@...wei.com>, Zhao Liu <zhao1.liu@...el.com>,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH v9 12/20] acpi/generic_event_device: add logic to detect
 if HEST addr is available

Em Mon, 2 Jun 2025 12:22:44 +0200
Igor Mammedov <imammedo@...hat.com> escreveu:

> On Fri, 30 May 2025 22:18:10 +0200
> Mauro Carvalho Chehab <mchehab+huawei@...nel.org> wrote:
> 
> > Em Fri, 30 May 2025 16:49:03 +0200
> > Igor Mammedov <imammedo@...hat.com> escreveu:
> >   
> > > On Fri, 30 May 2025 08:01:28 -0400
> > > "Michael S. Tsirkin" <mst@...hat.com> wrote:
> > >     
> > > > On Wed, May 28, 2025 at 05:42:12PM +0200, Igor Mammedov wrote:      
> > > > > On Tue, 20 May 2025 08:41:31 +0200
> > > > > Mauro Carvalho Chehab <mchehab+huawei@...nel.org> wrote:
> > > > >         
> > > > > > Create a new property (x-has-hest-addr) and use it to detect if
> > > > > > the GHES table offsets can be calculated from the HEST address
> > > > > > (qemu 10.0 and upper) or via the legacy way via an offset obtained
> > > > > > from the hardware_errors firmware file.        
> > > > > 
> > > > > 
> > > > > it doesn't apply to current master anymore        
> > > > 
> > > > indeed. Mauro?      
> > > 
> > > Michael,
> > > it's trivial conflict in machine compat,
> > > could you fix it up while applying?    
> > 
> > IMHO, that's the best. The thing is, as code gets merged upstream with
> > backports, conflicts happen.
> > 
> > I can re-send the series, if you prefer, as I'm keeping it rebasing it
> > from time to time at:
> > 	https://gitlab.com/mchehab_kernel/qemu/-/tree/qemu_submitted?ref_type=heads
> > 
> > (it is on the top of upstream/master)
> > 
> > But even that might have conflicts on your test tree if you pick
> > other patches touching this backport table:
> >   
> > > -GlobalProperty hw_compat_10_0[] = {};
> > > +GlobalProperty hw_compat_10_0[] = {
> > > +    { TYPE_ACPI_GED, "x-has-hest-addr", "false" },
> > > +};    
> > 
> > (this was the code when I sent the PR. When applying upstream,
> > such hunk is now(*):
> > 
> >  GlobalProperty hw_compat_10_0[] = {
> >      { "scsi-hd", "dpofua", "off" },
> > +    { TYPE_ACPI_GED, "x-has-hest-addr", "false" },
> >  };
> > 
> > 
> > (*) https://gitlab.com/mchehab_kernel/qemu/-/commit/08c4859f8c6f36d7dccf2b773be88847e5d1fe0c
> > 
> > If you still prefer that I resubmit the entire PR, let me know.  
> 
> If it's the only patch that needs rebase and doesn't affect the rest,
> I'd say there is no need to spam the list with whole series respin, 
> just post rebased v10 12/20 as reply here
> 
> If it's more than that, respin series.

This is the only patch with conflicts. I'll send a v10 of it.

> 
> > 
> > Regards,
> > Mauro
> >   
> 



Thanks,
Mauro