[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 08/10] tools: add physinfo arch_capabilities handling for Arm
- To: Luca Fancellu <luca.fancellu@xxxxxxx>, Edwin Torok <edvin.torok@xxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- From: Christian Lindig <christian.lindig@xxxxxxxxx>
- Date: Wed, 15 Mar 2023 09:50:08 +0000
- Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, wei.chen@xxxxxxx, George Dunlap <george.dunlap@xxxxxxxxxx>, Nick Rosbrook <rosbrookn@xxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Christian Lindig <christian.lindig@xxxxxxxxxx>, David Scott <dave@xxxxxxxxxx>, Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Wed, 15 Mar 2023 09:50:18 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
Acked-by: Christian Lindig <christian.lindig@xxxxxxxxx>
> On 15 Mar 2023, at 09:05, Luca Fancellu <luca.fancellu@xxxxxxx> wrote:
>
> type arch_physinfo_cap_flags =
> - | ARM of arm_physinfo_cap_flag list
> + | ARM of int
> | X86 of x86_physinfo_cap_flag list
>
I notice that other capabilities are split into a list and not kept as a bit
field. Is this not needed here or am I misunderstanding the representation?
— C
|
