From: Ani Sinha <ani@anisinha.ca>
To: "wangyanan (Y)" <wangyanan55@huawei.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
Andrew Jones <drjones@redhat.com>,
"Michael S . Tsirkin" <mst@redhat.com>,
qemu-devel@nongnu.org, Shannon Zhao <shannon.zhaosl@gmail.com>,
qemu-arm@nongnu.org, Eric Auger <eauger@redhat.com>,
wanghaibin.wang@huawei.com, Igor Mammedov <imammedo@redhat.com>
Subject: Re: [PATCH v6 7/7] tests/acpi/bios-table-test: Update expected virt/PPTT file
Date: Tue, 4 Jan 2022 09:57:52 +0530 (IST) [thread overview]
Message-ID: <alpine.OSX.2.20.2201040956570.24803@athabasca.local> (raw)
In-Reply-To: <d9c1574f-ab24-012f-0224-ab52eb7c0fc8@huawei.com>
On Tue, 4 Jan 2022, wangyanan (Y) wrote:
> Hi Ani,
> Thanks for your review.
>
> On 2022/1/3 20:01, Ani Sinha wrote:
> >
> > On Mon, 3 Jan 2022, Yanan Wang wrote:
> >
> > > Run ./tests/data/acpi/rebuild-expected-aml.sh from build directory
> > > to update PPTT binary. Also empty bios-tables-test-allowed-diff.h.
> > >
> > > The disassembled differences between actual and expected PPTT:
> > >
> > > /*
> > > * Intel ACPI Component Architecture
> > > * AML/ASL+ Disassembler version 20180810 (64-bit version)
> > > * Copyright (c) 2000 - 2018 Intel Corporation
> > > *
> > > - * Disassembly of tests/data/acpi/virt/PPTT, Mon Oct 25 20:24:53 2021
> > > + * Disassembly of /tmp/aml-BPI5B1, Mon Oct 25 20:24:53 2021
> > > *
> > > * ACPI Data Table [PPTT]
> > > *
> > > * Format: [HexOffset DecimalOffset ByteLength] FieldName : FieldValue
> > > */
> > >
> > > [000h 0000 4] Signature : "PPTT" [Processor
> > > Properties Topology Table]
> > > -[004h 0004 4] Table Length : 0000004C
> > > +[004h 0004 4] Table Length : 00000060
> > > [008h 0008 1] Revision : 02
> > > -[009h 0009 1] Checksum : A8
> > > +[009h 0009 1] Checksum : 48
> > > [00Ah 0010 6] Oem ID : "BOCHS "
> > > [010h 0016 8] Oem Table ID : "BXPC "
> > > [018h 0024 4] Oem Revision : 00000001
> > > [01Ch 0028 4] Asl Compiler ID : "BXPC"
> > > [020h 0032 4] Asl Compiler Revision : 00000001
> > >
> > > [024h 0036 1] Subtable Type : 00 [Processor Hierarchy
> > > Node]
> > > [025h 0037 1] Length : 14
> > > [026h 0038 2] Reserved : 0000
> > > [028h 0040 4] Flags (decoded below) : 00000001
> > > Physical package : 1
> > > ACPI Processor ID valid : 0
> > > [02Ch 0044 4] Parent : 00000000
> > > [030h 0048 4] ACPI Processor ID : 00000000
> > > [034h 0052 4] Private Resource Number : 00000000
> > >
> > > [038h 0056 1] Subtable Type : 00 [Processor Hierarchy
> > > Node]
> > > [039h 0057 1] Length : 14
> > > [03Ah 0058 2] Reserved : 0000
> > > -[03Ch 0060 4] Flags (decoded below) : 0000000A
> > > +[03Ch 0060 4] Flags (decoded below) : 00000000
> > > Physical package : 0
> > > - ACPI Processor ID valid : 1
> > > + ACPI Processor ID valid : 0
> > I do not know this very well but does the above two changes (flags and
> > processor ID) makes sense?
> Yes. I think this is exactly what we expected.
> Above flags is for the newly inserted cluster node which is between
> socket node and core node. Flag "Physical package" is 0 because
> it does not represent the boundary of physical package. Flag
> "ACPI Processor ID valid" is 0, because we don't need a valid ID
> for a container in QEMU (cluster is container of CPU core) just
> like socket node.
>
> "0000000A" originally comes from core node, which now is at
> place [*] below.
>
> We can also read the reason why we don't need a valid ID for a
> container in 099f2df2e6b "hw/acpi/aml-build: Add PPTT table".
Ok as long as we can explain it, I am good.
> >
> > > [040h 0064 4] Parent : 00000024
> > > [044h 0068 4] ACPI Processor ID : 00000000
> > > [048h 0072 4] Private Resource Number : 00000000
> > >
> > > -Raw Table Data: Length 76 (0x4C)
> > > +[04Ch 0076 1] Subtable Type : 00 [Processor Hierarchy
> > > Node]
> > > +[04Dh 0077 1] Length : 14
> > > +[04Eh 0078 2] Reserved : 0000
> > > +[050h 0080 4] Flags (decoded below) : 0000000A
> > > + Physical package : 0
> > > + ACPI Processor ID valid : 1
> > > +[054h 0084 4] Parent : 00000038
> > > +[058h 0088 4] ACPI Processor ID : 00000000
> > > +[05Ch 0092 4] Private Resource Number : 00000000
> [*] Information of core node.
>
> Thanks,
> Yanan
> > > +
> > > +Raw Table Data: Length 96 (0x60)
> > >
> > > - 0000: 50 50 54 54 4C 00 00 00 02 A8 42 4F 43 48 53 20 //
> > > PPTTL.....BOCHS
> > > + 0000: 50 50 54 54 60 00 00 00 02 48 42 4F 43 48 53 20 //
> > > PPTT`....HBOCHS
> > > 0010: 42 58 50 43 20 20 20 20 01 00 00 00 42 58 50 43 // BXPC
> > > ....BXPC
> > > 0020: 01 00 00 00 00 14 00 00 01 00 00 00 00 00 00 00 //
> > > ................
> > > - 0030: 00 00 00 00 00 00 00 00 00 14 00 00 0A 00 00 00 //
> > > ................
> > > - 0040: 24 00 00 00 00 00 00 00 00 00 00 00 //
> > > $...........
> > > + 0030: 00 00 00 00 00 00 00 00 00 14 00 00 00 00 00 00 //
> > > ................
> > > + 0040: 24 00 00 00 00 00 00 00 00 00 00 00 00 14 00 00 //
> > > $...............
> > > + 0050: 0A 00 00 00 38 00 00 00 00 00 00 00 00 00 00 00 //
> > > ....8...........
> > >
> > > Signed-off-by: Yanan Wang <wangyanan55@huawei.com>
Reviewed-by: Ani Sinha <ani@anisinha.ca>
> > > ---
> > > tests/data/acpi/virt/PPTT | Bin 76 -> 96 bytes
> > > tests/qtest/bios-tables-test-allowed-diff.h | 1 -
> > > 2 files changed, 1 deletion(-)
> > >
> > > diff --git a/tests/data/acpi/virt/PPTT b/tests/data/acpi/virt/PPTT
> > > index
> > > 7a1258ecf123555b24462c98ccbb76b4ac1d0c2b..f56ea63b369a604877374ad696c396e796ab1c83
> > > 100644
> > > GIT binary patch
> > > delta 53
> > > zcmV-50LuSNU<y!BR8(L90006=kqR;-00000Bme*a000000000002BZK3IG5AH~;_u
> > > L0000000000uCW9Z
> > >
> > > delta 32
> > > qcmV+*0N?*$ObSp?R8&j=00080kqR=APy`Gl00000000000001OcLdh}
> > >
> > > diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
> > > b/tests/qtest/bios-tables-test-allowed-diff.h
> > > index cb143a55a6..dfb8523c8b 100644
> > > --- a/tests/qtest/bios-tables-test-allowed-diff.h
> > > +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> > > @@ -1,2 +1 @@
> > > /* List of comma-separated changed AML files to ignore */
> > > -"tests/data/acpi/virt/PPTT",
> > > --
> > > 2.27.0
> > >
> > >
> > .
>
>
next prev parent reply other threads:[~2022-01-04 4:29 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-03 8:46 [PATCH v6 0/7] ARM virt: Support CPU cluster topology Yanan Wang via
2022-01-03 8:46 ` [PATCH v6 1/7] hw/arm/virt: Support CPU cluster on ARM virt machine Yanan Wang via
2022-01-03 11:24 ` Andrew Jones
2022-01-03 8:46 ` [PATCH v6 2/7] hw/arm/virt: Support cluster level in DT cpu-map Yanan Wang via
2022-01-03 11:25 ` Andrew Jones
2022-01-03 8:46 ` [PATCH v6 3/7] hw/acpi/aml-build: Improve scalability of PPTT generation Yanan Wang via
2022-01-03 11:24 ` Andrew Jones
2022-01-04 2:05 ` wangyanan (Y) via
2022-01-03 8:46 ` [PATCH v6 4/7] hw/arm/virt-acpi-build: Make an ARM specific PPTT generator Yanan Wang via
2022-01-03 11:30 ` Andrew Jones
2022-01-04 2:06 ` wangyanan (Y) via
2022-01-03 8:46 ` [PATCH v6 5/7] tests/acpi/bios-tables-test: Allow changes to virt/PPTT file Yanan Wang via
2022-01-03 11:51 ` Ani Sinha
2022-01-03 8:46 ` [PATCH v6 6/7] hw/arm/virt-acpi-build: Support cluster level in PPTT generation Yanan Wang via
2022-01-03 11:32 ` Andrew Jones
2022-01-04 2:15 ` wangyanan (Y) via
2022-01-03 8:46 ` [PATCH v6 7/7] tests/acpi/bios-table-test: Update expected virt/PPTT file Yanan Wang via
2022-01-03 12:01 ` Ani Sinha
2022-01-04 2:28 ` wangyanan (Y) via
2022-01-04 4:27 ` Ani Sinha [this message]
2022-01-04 4:51 ` wangyanan (Y) via
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.OSX.2.20.2201040956570.24803@athabasca.local \
--to=ani@anisinha.ca \
--cc=drjones@redhat.com \
--cc=eauger@redhat.com \
--cc=imammedo@redhat.com \
--cc=mst@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=shannon.zhaosl@gmail.com \
--cc=wanghaibin.wang@huawei.com \
--cc=wangyanan55@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).