Re: [PATCH 08/13] conf: Parse and format HMAT
On 7/3/20 11:51 AM, Michal Privoznik wrote:
On 7/3/20 11:23 AM, Daniel P. Berrangé wrote:
> On Wed, Jun 24, 2020 at 03:49:06PM +0200, Michal Privoznik wrote:
>> To cite ACPI specification:
>>
>
> Functionally the XML looks find to me, but I was just thinking it looks
> a bit wierd to have <bandwidth> under a <latencies> wrapper. I'm not
> entirely sure what better name we should use - perhaps "interconnects" ?
>
> <interconnects>
> <latency initiator='0' target='0' type='access'
value='5'/>
> <latency initiator='0' target='0' cache='1'
type='access'
> value='10'/>
> <bandwidth initiator='0' target='0'
type='access'
> value='204800' unit='KiB'/>
> </interconnects>
>
> any other ideas ?
That sounds better. Or we can admit this is HMAT an have <hmat/> instead
<interconnects/>? But that won't be much future proof so your suggestion
sound better.
Alright, no one suggested anything better and I'd like to push these to
get the most out of the development window possible. I'll change this to
<interconnects/> and push. We can still change it until the release, if
needed.
Michal