[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/3] megasas: use unsigned type for positive numeric field
From: |
Darren Kenny |
Subject: |
Re: [PATCH v2 3/3] megasas: use unsigned type for positive numeric fields |
Date: |
Thu, 14 May 2020 14:28:44 +0100 |
On Thursday, 2020-05-14 at 00:55:40 +0530, P J P wrote:
> From: Prasad J Pandit <address@hidden>
>
> Use unsigned type for the MegasasState fields which hold positive
> numeric values.
>
> Signed-off-by: Prasad J Pandit <address@hidden>
Reviewed-by: Darren Kenny <address@hidden>
> ---
> hw/scsi/megasas.c | 38 +++++++++++++++++++-------------------
> 1 file changed, 19 insertions(+), 19 deletions(-)
>
> diff --git a/hw/scsi/megasas.c b/hw/scsi/megasas.c
> index b531d88a9b..634af0bbb8 100644
> --- a/hw/scsi/megasas.c
> +++ b/hw/scsi/megasas.c
> @@ -86,34 +86,34 @@ typedef struct MegasasState {
> MemoryRegion queue_io;
> uint32_t frame_hi;
>
> - int fw_state;
> + uint32_t fw_state;
> uint32_t fw_sge;
> uint32_t fw_cmds;
> uint32_t flags;
> - int fw_luns;
> - int intr_mask;
> - int doorbell;
> - int busy;
> - int diag;
> - int adp_reset;
> + uint32_t fw_luns;
> + uint32_t intr_mask;
> + uint32_t doorbell;
> + uint32_t busy;
> + uint32_t diag;
> + uint32_t adp_reset;
> OnOffAuto msi;
> OnOffAuto msix;
>
> MegasasCmd *event_cmd;
> - int event_locale;
> + uint16_t event_locale;
> int event_class;
> - int event_count;
> - int shutdown_event;
> - int boot_event;
> + uint32_t event_count;
> + uint32_t shutdown_event;
> + uint32_t boot_event;
>
> uint64_t sas_addr;
> char *hba_serial;
>
> uint64_t reply_queue_pa;
> void *reply_queue;
> - int reply_queue_len;
> + uint16_t reply_queue_len;
> uint16_t reply_queue_head;
> - int reply_queue_tail;
> + uint16_t reply_queue_tail;
> uint64_t consumer_pa;
> uint64_t producer_pa;
>
> @@ -2259,9 +2259,9 @@ static const VMStateDescription vmstate_megasas_gen1 = {
> VMSTATE_PCI_DEVICE(parent_obj, MegasasState),
> VMSTATE_MSIX(parent_obj, MegasasState),
>
> - VMSTATE_INT32(fw_state, MegasasState),
> - VMSTATE_INT32(intr_mask, MegasasState),
> - VMSTATE_INT32(doorbell, MegasasState),
> + VMSTATE_UINT32(fw_state, MegasasState),
> + VMSTATE_UINT32(intr_mask, MegasasState),
> + VMSTATE_UINT32(doorbell, MegasasState),
> VMSTATE_UINT64(reply_queue_pa, MegasasState),
> VMSTATE_UINT64(consumer_pa, MegasasState),
> VMSTATE_UINT64(producer_pa, MegasasState),
> @@ -2278,9 +2278,9 @@ static const VMStateDescription vmstate_megasas_gen2 = {
> VMSTATE_PCI_DEVICE(parent_obj, MegasasState),
> VMSTATE_MSIX(parent_obj, MegasasState),
>
> - VMSTATE_INT32(fw_state, MegasasState),
> - VMSTATE_INT32(intr_mask, MegasasState),
> - VMSTATE_INT32(doorbell, MegasasState),
> + VMSTATE_UINT32(fw_state, MegasasState),
> + VMSTATE_UINT32(intr_mask, MegasasState),
> + VMSTATE_UINT32(doorbell, MegasasState),
> VMSTATE_UINT64(reply_queue_pa, MegasasState),
> VMSTATE_UINT64(consumer_pa, MegasasState),
> VMSTATE_UINT64(producer_pa, MegasasState),
> --
> 2.25.4