[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] hmat acpi: Don't require initiator value in -numa when hmat=
From: |
Jonathan Cameron |
Subject: |
Re: [PATCH] hmat acpi: Don't require initiator value in -numa when hmat=on |
Date: |
Thu, 23 Jun 2022 12:24:57 +0100 |
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