[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 18/43] apic-common: make "id" property a uint
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v3 18/43] apic-common: make "id" property a uint32 |
Date: |
Thu, 08 Jun 2017 18:39:33 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Marc-André Lureau <address@hidden> writes:
> The getter and setter of TYPE_APIC_COMMON property "id" are
> apic_common_get_id() and apic_common_set_id().
>
> apic_common_get_id() reads either APICCommonState member uint32_t
> initial_apic_id or uint8_t id into an int64_t local variable. It then
> passes this variable to visit_type_int().
>
> apic_common_set_id() uses visit_type_int() to read the value into a
> local variable, which it then assigns both to initial_apic_id and id.
>
> While the state backing the property is two unsigned members, 8 and 32
> bits wide, the actual visitor is 64 bits signed.
>
> Change getter and setter to use visit_type_uint32(). Then everything's
> uint32_t, except for @id.
>
> Suggested-by: Markus Armbruster <address@hidden>
> Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
- [Qemu-devel] [PATCH v3 13/43] q35: fix get_mmcfg_size to use uint64 visitor, (continued)
- [Qemu-devel] [PATCH v3 16/43] qdev: wrap default property value in an union, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 17/43] qdev: avoid type casts between signed and unsigned, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 18/43] apic-common: make "id" property a uint32, Marc-André Lureau, 2017/06/07
- Re: [Qemu-devel] [PATCH v3 18/43] apic-common: make "id" property a uint32,
Markus Armbruster <=
- [Qemu-devel] [PATCH v3 19/43] qdev: use appropriate getter/setters type, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 20/43] isa: use get_uint() for "io-base", Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 21/43] pc-dimm: use get_uint() for dimm properties, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 22/43] pc-dimm: make "size" property uint64, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 23/43] pcihp: use get_uint() for "bsel" property, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 24/43] aspeed: use {set, get}_uint() for "ram-size" property, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 25/43] bcm2835_fb: use {get, set}_uint() for "vcram-size" and "vcram-base", Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 26/43] platform-bus: use get_uint() for "addr" property, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH v3 27/43] acpi: use get_uint() for "acpi-pcihp-io*" properties, Marc-André Lureau, 2017/06/07