On Tue, Sep 29, 2020 at 04:20:15PM +0530, Ani Sinha wrote: > as > > On Tue, Sep 29, 2020 at 4:11 PM Ani Sinha <[email protected]> wrote: > > > > On Tue, Sep 29, 2020 at 4:07 PM Michael S. Tsirkin <[email protected]> wrote: > > > > > > On Tue, Sep 29, 2020 at 04:02:07PM +0530, Ani Sinha wrote: > > > > On Tue, Sep 29, 2020 at 4:00 PM Ani Sinha <[email protected]> wrote: > > > > > > > > > > In your pull request the following patch is completely screwed up: > > > > > > > > > > commit cda2006eded0ed91974e1d9e7f9f288e65812a3e > > > > > Author: Ani Sinha <[email protected]> > > > > > Date: Tue Sep 29 03:22:52 2020 -0400 > > > > > > > > > > tests/acpi: update golden master DSDT binary table blobs for q35 > > > > > > > > > > > > > > > This is not my patch. It has all sorts of changes which does not > > > > > belong there. Can you please check? > > > > > > > > See > > > > https://patchew.org/QEMU/[email protected]/[email protected]/ > > > > > > > > > I had to regenerate the binary, yes. That's par for the course. > > > But it looks like I added disasssembled files. Will fix up and drop, > > > thanks for noticing this. > > It's probably worthwhile to regenerate DSDT.hpbrroot as well and then > do the diff and compare.
They are all regenerated by tests/data/acpi/rebuild-expected-aml.sh What is important is to check the changes and not trust them blindly. Do you see anything unexpected? It is easy to run tests/data/acpi/disassemle-aml.sh -o <output directory> on multiple versions and see what is going on. Care to do this? -- MST
