[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 04/20] nubus: use bitmap to manage available slots
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 04/20] nubus: use bitmap to manage available slots |
Date: |
Wed, 15 Sep 2021 12:39:37 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 |
On 9/15/21 12:10 PM, Mark Cave-Ayland wrote:
> Convert nubus_device_realize() to use a bitmap to manage available slots to
> allow
> for future Nubus devices to be plugged into arbitrary slots from the command
> line.
>
> Update mac_nubus_bridge_init() to only allow slots 0x9 to 0xe on a Macintosh
> machines as documented in "Desigining Cards and Drivers for the Macintosh
> Family".
>
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> ---
> hw/nubus/mac-nubus-bridge.c | 3 +++
> hw/nubus/nubus-bus.c | 2 +-
> hw/nubus/nubus-device.c | 33 +++++++++++++++++++++++++++------
> include/hw/nubus/nubus.h | 4 ++--
> 4 files changed, 33 insertions(+), 9 deletions(-)
> static void nubus_class_init(ObjectClass *oc, void *data)
> diff --git a/hw/nubus/nubus-device.c b/hw/nubus/nubus-device.c
> index c1832f73da..f9f614cc01 100644
> --- a/hw/nubus/nubus-device.c
> +++ b/hw/nubus/nubus-device.c
> @@ -160,14 +160,35 @@ static void nubus_device_realize(DeviceState *dev,
> Error **errp)
> NubusDevice *nd = NUBUS_DEVICE(dev);
> char *name;
> hwaddr slot_offset;
> -
> - if (nubus->current_slot < NUBUS_FIRST_SLOT ||
> - nubus->current_slot > NUBUS_LAST_SLOT) {
> - error_setg(errp, "Cannot register nubus card, not enough slots");
> - return;
> + uint16_t s;
> +
> + if (nd->slot == -1) {
> + /* No slot specified, find first available free slot */
> + s = ctz32(nubus->slot_available_mask);
> + if (s != 32) {
> + nd->slot = s;
Can't we have s >= NUBUS_LAST_SLOT here?
> + } else {
> + error_setg(errp, "Cannot register nubus card, no free slot "
> + "available");
> + return;
> + }
> + } else {
If so, maybe better to move this code <...
> + /* Slot specified, make sure the slot is available */
> + if (nd->slot < NUBUS_FIRST_SLOT || nd->slot > NUBUS_LAST_SLOT) {
> + error_setg(errp, "Cannot register nubus card, slot must be "
> + "between %d and %d", NUBUS_FIRST_SLOT,
> + NUBUS_LAST_SLOT);
> + return;
> + }
...> ...
> +
> + if (!(nubus->slot_available_mask & (1UL << nd->slot))) {
> + error_setg(errp, "Cannot register nubus card, slot %d is "
> + "unavailable or already occupied", nd->slot);
> + return;
> + }
> }
... here?
>
> - nd->slot = nubus->current_slot++;
> + nubus->slot_available_mask &= ~(1UL << nd->slot);
>
> /* Super */
> slot_offset = (nd->slot - 6) * NUBUS_SUPER_SLOT_SIZE;
> diff --git a/include/hw/nubus/nubus.h b/include/hw/nubus/nubus.h
> index 357f621d15..8ff4736259 100644
> --- a/include/hw/nubus/nubus.h
> +++ b/include/hw/nubus/nubus.h
> @@ -19,7 +19,7 @@
> #define NUBUS_SLOT_SIZE 0x01000000
> #define NUBUS_SLOT_NB 0xF
>
> -#define NUBUS_FIRST_SLOT 0x9
> +#define NUBUS_FIRST_SLOT 0x0
> #define NUBUS_LAST_SLOT 0xF
>
> #define TYPE_NUBUS_DEVICE "nubus-device"
> @@ -36,7 +36,7 @@ struct NubusBus {
> MemoryRegion super_slot_io;
> MemoryRegion slot_io;
>
> - int current_slot;
> + uint32_t slot_available_mask;
> };
>
> struct NubusDevice {
>
- [PATCH v2 00/20] nubus: bus, device, bridge, IRQ and address space improvements, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 01/20] nubus-device: rename slot_nb variable to slot, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 02/20] nubus-device: expose separate super slot memory region, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 03/20] nubus-device: add device slot parameter, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 05/20] nubus: move slot bitmap checks from NubusDevice realize() to BusClass check_address(), Mark Cave-Ayland, 2021/09/15
- [PATCH v2 06/20] nubus: implement BusClass get_dev_path(), Mark Cave-Ayland, 2021/09/15
- [PATCH v2 07/20] nubus: add trace-events for empty slot accesses, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 04/20] nubus: use bitmap to manage available slots, Mark Cave-Ayland, 2021/09/15
- Re: [PATCH v2 04/20] nubus: use bitmap to manage available slots,
Philippe Mathieu-Daudé <=
- [PATCH v2 08/20] nubus: generate bus error when attempting to access empty slots, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 09/20] macfb: don't register declaration ROM, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 10/20] nubus-device: remove nubus_register_rom() and nubus_register_format_block(), Mark Cave-Ayland, 2021/09/15
- [PATCH v2 11/20] nubus-device: add romfile property for loading declaration ROMs, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 12/20] nubus: move nubus to its own 32-bit address space, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 13/20] nubus-bridge: introduce separate NubusBridge structure, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 14/20] mac-nubus-bridge: rename MacNubusState to MacNubusBridge, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 15/20] nubus: move NubusBus from mac-nubus-bridge to nubus-bridge, Mark Cave-Ayland, 2021/09/15
- [PATCH v2 17/20] nubus-bridge: make slot_available_mask a qdev property, Mark Cave-Ayland, 2021/09/15