* [Qemu-arm] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
@ 2019-07-01 19:08 ` Eric Auger
0 siblings, 0 replies; 11+ messages in thread
From: Eric Auger @ 2019-07-01 19:08 UTC (permalink / raw)
To: eric.auger.pro, eric.auger, qemu-devel, qemu-arm, peter.maydell,
shameerali.kolothum.thodi, imammedo
PCDIMM hotplug addition updated the DSDT. Update the reference table.
Signed-off-by: Eric Auger <eric.auger@redhat.com>
---
tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
1 file changed, 0 insertions(+), 0 deletions(-)
diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
GIT binary patch
delta 172
zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
Ni>WOjJjjWw0RY`tDw+TQ
delta 134
zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
--
2.20.1
^ permalink raw reply [flat|nested] 11+ messages in thread* [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
@ 2019-07-01 19:08 ` Eric Auger
0 siblings, 0 replies; 11+ messages in thread
From: Eric Auger @ 2019-07-01 19:08 UTC (permalink / raw)
To: eric.auger.pro, eric.auger, qemu-devel, qemu-arm, peter.maydell,
shameerali.kolothum.thodi, imammedo
PCDIMM hotplug addition updated the DSDT. Update the reference table.
Signed-off-by: Eric Auger <eric.auger@redhat.com>
---
tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
1 file changed, 0 insertions(+), 0 deletions(-)
diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
GIT binary patch
delta 172
zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
Ni>WOjJjjWw0RY`tDw+TQ
delta 134
zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
--
2.20.1
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-arm] [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
2019-07-01 19:08 ` [Qemu-devel] " Eric Auger
@ 2019-07-02 8:56 ` Philippe Mathieu-Daudé
-1 siblings, 0 replies; 11+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-07-02 8:56 UTC (permalink / raw)
To: Eric Auger, eric.auger.pro, qemu-devel, qemu-arm, peter.maydell,
shameerali.kolothum.thodi, imammedo, David Hildenbrand
Hi Eric,
On 7/1/19 9:08 PM, Eric Auger wrote:
> PCDIMM hotplug addition updated the DSDT. Update the reference table.
Can you point the commit sha1 please?
> Signed-off-by: Eric Auger <eric.auger@redhat.com>
> ---
> tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
> 1 file changed, 0 insertions(+), 0 deletions(-)
>
> diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
> index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
> GIT binary patch
> delta 172
> zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
> z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
> zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
> Ni>WOjJjjWw0RY`tDw+TQ
>
> delta 134
> zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
> zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
> Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
>
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
@ 2019-07-02 8:56 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 11+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-07-02 8:56 UTC (permalink / raw)
To: Eric Auger, eric.auger.pro, qemu-devel, qemu-arm, peter.maydell,
shameerali.kolothum.thodi, imammedo, David Hildenbrand
Hi Eric,
On 7/1/19 9:08 PM, Eric Auger wrote:
> PCDIMM hotplug addition updated the DSDT. Update the reference table.
Can you point the commit sha1 please?
> Signed-off-by: Eric Auger <eric.auger@redhat.com>
> ---
> tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
> 1 file changed, 0 insertions(+), 0 deletions(-)
>
> diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
> index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
> GIT binary patch
> delta 172
> zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
> z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
> zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
> Ni>WOjJjjWw0RY`tDw+TQ
>
> delta 134
> zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
> zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
> Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
>
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-arm] [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
2019-07-02 8:56 ` Philippe Mathieu-Daudé
@ 2019-07-02 9:01 ` Auger Eric
-1 siblings, 0 replies; 11+ messages in thread
From: Auger Eric @ 2019-07-02 9:01 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, eric.auger.pro, qemu-devel, qemu-arm,
peter.maydell, shameerali.kolothum.thodi, imammedo,
David Hildenbrand
Hi Philippe,
On 7/2/19 10:56 AM, Philippe Mathieu-Daudé wrote:
> Hi Eric,
>
> On 7/1/19 9:08 PM, Eric Auger wrote:
>> PCDIMM hotplug addition updated the DSDT. Update the reference table.
>
> Can you point the commit sha1 please?
this follows Peter's make check failure report sent yesterday and
follow-up discussion on IRC. The series is not yet upstream so I cannot
point to any SHA1 yet.
https://www.mail-archive.com/qemu-devel@nongnu.org/msg627971.html
Thanks
Eric
>
>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>> ---
>> tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
>> 1 file changed, 0 insertions(+), 0 deletions(-)
>>
>> diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
>> index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
>> GIT binary patch
>> delta 172
>> zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
>> z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
>> zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
>> Ni>WOjJjjWw0RY`tDw+TQ
>>
>> delta 134
>> zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
>> zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
>> Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
>>
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
@ 2019-07-02 9:01 ` Auger Eric
0 siblings, 0 replies; 11+ messages in thread
From: Auger Eric @ 2019-07-02 9:01 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, eric.auger.pro, qemu-devel, qemu-arm,
peter.maydell, shameerali.kolothum.thodi, imammedo,
David Hildenbrand
Hi Philippe,
On 7/2/19 10:56 AM, Philippe Mathieu-Daudé wrote:
> Hi Eric,
>
> On 7/1/19 9:08 PM, Eric Auger wrote:
>> PCDIMM hotplug addition updated the DSDT. Update the reference table.
>
> Can you point the commit sha1 please?
this follows Peter's make check failure report sent yesterday and
follow-up discussion on IRC. The series is not yet upstream so I cannot
point to any SHA1 yet.
https://www.mail-archive.com/qemu-devel@nongnu.org/msg627971.html
Thanks
Eric
>
>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>> ---
>> tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
>> 1 file changed, 0 insertions(+), 0 deletions(-)
>>
>> diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
>> index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
>> GIT binary patch
>> delta 172
>> zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
>> z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
>> zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
>> Ni>WOjJjjWw0RY`tDw+TQ
>>
>> delta 134
>> zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
>> zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
>> Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
>>
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-arm] [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
2019-07-02 9:01 ` Auger Eric
@ 2019-07-02 9:43 ` Philippe Mathieu-Daudé
-1 siblings, 0 replies; 11+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-07-02 9:43 UTC (permalink / raw)
To: Auger Eric, eric.auger.pro, qemu-devel, qemu-arm, peter.maydell,
shameerali.kolothum.thodi, imammedo, David Hildenbrand
On 7/2/19 11:01 AM, Auger Eric wrote:
> Hi Philippe,
> On 7/2/19 10:56 AM, Philippe Mathieu-Daudé wrote:
>> Hi Eric,
>>
>> On 7/1/19 9:08 PM, Eric Auger wrote:
>>> PCDIMM hotplug addition updated the DSDT. Update the reference table.
>>
>> Can you point the commit sha1 please?
>
> this follows Peter's make check failure report sent yesterday and
> follow-up discussion on IRC. The series is not yet upstream so I cannot
> point to any SHA1 yet.
>
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg627971.html
I hadn't review this series, so this patch was making no sense at all to
me, now it does.
I think it has to be squashed with:
"hw/arm/virt: Enable device memory cold/hot plug with ACPI boot"
in the series you mentioned.
Regards,
Phil.
>>
>>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>>> ---
>>> tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
>>> 1 file changed, 0 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
>>> index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
>>> GIT binary patch
>>> delta 172
>>> zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
>>> z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
>>> zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
>>> Ni>WOjJjjWw0RY`tDw+TQ
>>>
>>> delta 134
>>> zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
>>> zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
>>> Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
>>>
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
@ 2019-07-02 9:43 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 11+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-07-02 9:43 UTC (permalink / raw)
To: Auger Eric, eric.auger.pro, qemu-devel, qemu-arm, peter.maydell,
shameerali.kolothum.thodi, imammedo, David Hildenbrand
On 7/2/19 11:01 AM, Auger Eric wrote:
> Hi Philippe,
> On 7/2/19 10:56 AM, Philippe Mathieu-Daudé wrote:
>> Hi Eric,
>>
>> On 7/1/19 9:08 PM, Eric Auger wrote:
>>> PCDIMM hotplug addition updated the DSDT. Update the reference table.
>>
>> Can you point the commit sha1 please?
>
> this follows Peter's make check failure report sent yesterday and
> follow-up discussion on IRC. The series is not yet upstream so I cannot
> point to any SHA1 yet.
>
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg627971.html
I hadn't review this series, so this patch was making no sense at all to
me, now it does.
I think it has to be squashed with:
"hw/arm/virt: Enable device memory cold/hot plug with ACPI boot"
in the series you mentioned.
Regards,
Phil.
>>
>>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>>> ---
>>> tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
>>> 1 file changed, 0 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
>>> index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
>>> GIT binary patch
>>> delta 172
>>> zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
>>> z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
>>> zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
>>> Ni>WOjJjjWw0RY`tDw+TQ
>>>
>>> delta 134
>>> zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
>>> zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
>>> Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
>>>
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-arm] [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
2019-07-02 9:43 ` Philippe Mathieu-Daudé
@ 2019-07-02 9:52 ` Auger Eric
-1 siblings, 0 replies; 11+ messages in thread
From: Auger Eric @ 2019-07-02 9:52 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, eric.auger.pro, qemu-devel, qemu-arm,
peter.maydell, shameerali.kolothum.thodi, imammedo,
David Hildenbrand
Hi Philippe,
On 7/2/19 11:43 AM, Philippe Mathieu-Daudé wrote:
> On 7/2/19 11:01 AM, Auger Eric wrote:
>> Hi Philippe,
>> On 7/2/19 10:56 AM, Philippe Mathieu-Daudé wrote:
>>> Hi Eric,
>>>
>>> On 7/1/19 9:08 PM, Eric Auger wrote:
>>>> PCDIMM hotplug addition updated the DSDT. Update the reference table.
>>>
>>> Can you point the commit sha1 please?
>>
>> this follows Peter's make check failure report sent yesterday and
>> follow-up discussion on IRC. The series is not yet upstream so I cannot
>> point to any SHA1 yet.
>>
>> https://www.mail-archive.com/qemu-devel@nongnu.org/msg627971.html
>
> I hadn't review this series, so this patch was making no sense at all to
> me, now it does.
>
> I think it has to be squashed with:
> "hw/arm/virt: Enable device memory cold/hot plug with ACPI boot"
> in the series you mentioned.
Yep sorry for that. I was trying to fix the issue quickly for unblocking
the series for 4.1.
Actually it looks really weird as the test is passing on some hosts
(Igor is currently experiencing it). At the beginning I was not able to
reproduce Peter's issue and eventually after many various tweaks in the
environment I was able to. I used the script to regenerate the DSDT and
this definitively fixes the issue for me. Now we are not sure this isn't
an issue in the test itself.
Thanks
Eric
>
> Regards,
>
> Phil.
>
>>>
>>>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>>>> ---
>>>> tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
>>>> 1 file changed, 0 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
>>>> index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
>>>> GIT binary patch
>>>> delta 172
>>>> zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
>>>> z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
>>>> zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
>>>> Ni>WOjJjjWw0RY`tDw+TQ
>>>>
>>>> delta 134
>>>> zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
>>>> zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
>>>> Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
>>>>
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
@ 2019-07-02 9:52 ` Auger Eric
0 siblings, 0 replies; 11+ messages in thread
From: Auger Eric @ 2019-07-02 9:52 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, eric.auger.pro, qemu-devel, qemu-arm,
peter.maydell, shameerali.kolothum.thodi, imammedo,
David Hildenbrand
Hi Philippe,
On 7/2/19 11:43 AM, Philippe Mathieu-Daudé wrote:
> On 7/2/19 11:01 AM, Auger Eric wrote:
>> Hi Philippe,
>> On 7/2/19 10:56 AM, Philippe Mathieu-Daudé wrote:
>>> Hi Eric,
>>>
>>> On 7/1/19 9:08 PM, Eric Auger wrote:
>>>> PCDIMM hotplug addition updated the DSDT. Update the reference table.
>>>
>>> Can you point the commit sha1 please?
>>
>> this follows Peter's make check failure report sent yesterday and
>> follow-up discussion on IRC. The series is not yet upstream so I cannot
>> point to any SHA1 yet.
>>
>> https://www.mail-archive.com/qemu-devel@nongnu.org/msg627971.html
>
> I hadn't review this series, so this patch was making no sense at all to
> me, now it does.
>
> I think it has to be squashed with:
> "hw/arm/virt: Enable device memory cold/hot plug with ACPI boot"
> in the series you mentioned.
Yep sorry for that. I was trying to fix the issue quickly for unblocking
the series for 4.1.
Actually it looks really weird as the test is passing on some hosts
(Igor is currently experiencing it). At the beginning I was not able to
reproduce Peter's issue and eventually after many various tweaks in the
environment I was able to. I used the script to regenerate the DSDT and
this definitively fixes the issue for me. Now we are not sure this isn't
an issue in the test itself.
Thanks
Eric
>
> Regards,
>
> Phil.
>
>>>
>>>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>>>> ---
>>>> tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
>>>> 1 file changed, 0 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
>>>> index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
>>>> GIT binary patch
>>>> delta 172
>>>> zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
>>>> z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
>>>> zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
>>>> Ni>WOjJjjWw0RY`tDw+TQ
>>>>
>>>> delta 134
>>>> zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
>>>> zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
>>>> Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
>>>>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes
2019-07-01 19:08 ` [Qemu-devel] " Eric Auger
(?)
(?)
@ 2019-07-02 10:59 ` Igor Mammedov
-1 siblings, 0 replies; 11+ messages in thread
From: Igor Mammedov @ 2019-07-02 10:59 UTC (permalink / raw)
To: Eric Auger
Cc: peter.maydell, qemu-arm, qemu-devel, shameerali.kolothum.thodi,
eric.auger.pro
On Mon, 1 Jul 2019 21:08:19 +0200
Eric Auger <eric.auger@redhat.com> wrote:
> PCDIMM hotplug addition updated the DSDT. Update the reference table.
>
> Signed-off-by: Eric Auger <eric.auger@redhat.com>
checked ASL after applying it.
Modulo minor issue with generated ASL (reference to non existing method when running without memory hotplug enabled) which lax linux AML interpreter happily ignores if method is not actually accessed.
(could be fixed later on top of Shameer's series as trivial bugfix)
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
> ---
> tests/data/acpi/virt/DSDT | Bin 18476 -> 18493 bytes
> 1 file changed, 0 insertions(+), 0 deletions(-)
>
> diff --git a/tests/data/acpi/virt/DSDT b/tests/data/acpi/virt/DSDT
> index 20e85c7f89f645c69935c615c07084e221419960..1a5c3e5243c78eb25345fdf7ffa442afec3c5c28 100644
> GIT binary patch
> delta 172
> zcmZ28fpPByMlP3Nmk?VI1_q{Y6S<_BIvO{s=eSSi^^mWRZt~=a(Tfjuig$N)iRXy-
> z@O0sIbPn(|FfcS`;D`?ea)A;+mUB?BARiZZCmRDZqb36bLnT9WgKLnhI|B~`0|zG;
> zOLQYQi0kYc?CPT+ViNBf7Q$Ho;v_5(n_Qj1$iNt*&kVB1*CW8$H`v*4fdEjLhl`1U
> Ni>WOjJjjWw0RY`tDw+TQ
>
> delta 134
> zcmdlxfpN_QMlP3Nmk=Ef1_q{ziCof5?F}2%bKIpo*xdvC4LIUGJY9GlgM2*<49pA}
> zIN}{$f+og`c`$P_2rw{ovN13+sxvS!RDxt&Jq4Ay6ndl?7@-g-A<n=c$sh-0Ff)LR
> Z11c2ZjdwLLW@rlt4|3vam^{&45den68LI#Q
>
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2019-07-02 11:08 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-07-01 19:08 [Qemu-arm] [PATCH] tests: Update DSDT ACPI table for arm/virt board with PCDIMM related changes Eric Auger
2019-07-01 19:08 ` [Qemu-devel] " Eric Auger
2019-07-02 8:56 ` [Qemu-arm] " Philippe Mathieu-Daudé
2019-07-02 8:56 ` Philippe Mathieu-Daudé
2019-07-02 9:01 ` [Qemu-arm] " Auger Eric
2019-07-02 9:01 ` Auger Eric
2019-07-02 9:43 ` [Qemu-arm] " Philippe Mathieu-Daudé
2019-07-02 9:43 ` Philippe Mathieu-Daudé
2019-07-02 9:52 ` [Qemu-arm] " Auger Eric
2019-07-02 9:52 ` Auger Eric
2019-07-02 10:59 ` Igor Mammedov
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.