Re: [PATCH] hmat acpi: Don't require initiator value in -numa when hmat=on

[Jonathan Cameron via <qemu-devel@nongnu.org>]

On Tue, 21 Jun 2022 12:49:09 +0200
Brice Goglin <Brice.Goglin@inria.fr> wrote:

> Le 20/06/2022 à 18:05, Igor Mammedov a écrit :
> > On Mon, 20 Jun 2022 17:24:18 +0200
> > Brice Goglin <Brice.Goglin@inria.fr> wrote:
> >  
> >> Le 20/06/2022 à 15:27, Igor Mammedov a écrit Machine (2966MB total) +
> >> Package P#0  
> >>>>      NUMANode P#2 (979MB)
> >>>>      Group0
> >>>>        NUMANode P#0 (980MB)
> >>>>        Core P#0 + PU P#0
> >>>>        Core P#1 + PU P#1
> >>>>      Group0
> >>>>        NUMANode P#1 (1007MB)
> >>>>        Core P#2 + PU P#2
> >>>>        Core P#3 + PU P#3  
> >>> here should be a dis-assembled dump of generated HMAT table  
> >>
> >> Hello
> >>
> >> Like what I added at the end of
> >> https://github.com/bgoglin/qemu/commit/d9b3f5cb1514adafa644afcc2a363f2dc9795a32
> >> ?  
> > yep, only full version including headers.  
> 
> 
> Thanks,
> 
> I split the patch adding the new test in 3 patches, reduced memory to 
> 128M total, etc.
> 
> I am not sure I understood what I am supposed to put in commit messages. 
> Can you check the 4 patches on top of 
> https://github.com/bgoglin/qemu/commits/hmat-noinitiator before I resend 
> them to the list?
> 
> Brice
> 

I left a couple of trivial comments on github, but basically looks good
to me.

Jonathan