linux-kernel - Re: [PATCH v6 17/19] tests/acpi: virt: update HEST and DSDT tables

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]

Message-ID: <20250228140439.450611bd@imammedo.users.ipa.redhat.com>
Date: Fri, 28 Feb 2025 14:04:39 +0100
From: Igor Mammedov <imammedo@...hat.com>
To: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
Cc: Jonathan Cameron <Jonathan.Cameron@...wei.com>, "Michael S . Tsirkin"
 <mst@...hat.com>, Shiju Jose <shiju.jose@...wei.com>,
 <qemu-arm@...gnu.org>, <qemu-devel@...gnu.org>, Ani Sinha
 <anisinha@...hat.com>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v6 17/19] tests/acpi: virt: update HEST and DSDT tables

On Fri, 28 Feb 2025 10:44:22 +0100
Mauro Carvalho Chehab <mchehab+huawei@...nel.org> wrote:

> Em Fri, 28 Feb 2025 17:33:18 +0800
> Jonathan Cameron <Jonathan.Cameron@...wei.com> escreveu:
> 
> > On Thu, 27 Feb 2025 17:00:55 +0100
> > Mauro Carvalho Chehab <mchehab+huawei@...nel.org> wrote:
> >   
> > > - The HEST table now accept two sources;
> > > - The DSDT tables now have a GED error device.
> > > 
> > > @@ -1,39 +1,39 @@
> > >  /*
> > >   * Intel ACPI Component Architecture
> > >   * AML/ASL+ Disassembler version 20240322 (64-bit version)
> > >   * Copyright (c) 2000 - 2023 Intel Corporation
> > >   *
> > > - * Disassembly of tests/data/acpi/aarch64/virt/HEST
> > > + * Disassembly of /tmp/aml-DMPE22    
> > This is an artifact of where you happened to get file
> > from so if we are being really fussy drop the change 
> > diff in the patch description for it.  I don't really care though.  
> > >   *    
> 
> True, but on the other hand, this is the only place at the diff
> saying what table the diff is against (HEST, in this case), as I had to
> remove the information before @@, as it was causing troubles when Igor
> were trying to apply the patch (plus on checkpatch).
> 
> That' why I opted to keep this hunk.

I'd drop this up to '* ACPI Data Table [HEST]' line.
But doesn't really matter, as long as reader can get what 's going on.
(there were precedents just putting snippets of diff)

Acked-by: Igor Mammedov <imammedo@...hat.com>

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