[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH-for-9.1 v2 08/11] hw/net/lan9118: Use RX_STATUS_FIFO_BYTES de
From: |
Peter Maydell |
Subject: |
Re: [PATCH-for-9.1 v2 08/11] hw/net/lan9118: Use RX_STATUS_FIFO_BYTES definition |
Date: |
Tue, 9 Apr 2024 14:59:55 +0100 |
On Tue, 9 Apr 2024 at 14:39, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> rx_status_fifo[] is an array of words,
> rx_status_fifo_size is a word count.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> hw/net/lan9118.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/hw/net/lan9118.c b/hw/net/lan9118.c
> index a983ce193b..cace22381d 100644
> --- a/hw/net/lan9118.c
> +++ b/hw/net/lan9118.c
> @@ -262,7 +262,7 @@ struct lan9118_state {
> int32_t rx_status_fifo_size;
> int32_t rx_status_fifo_used;
> int32_t rx_status_fifo_head;
> - uint32_t rx_status_fifo[896];
> + uint32_t rx_status_fifo[RX_STATUS_FIFO_BYTES / 4];
> int32_t rx_fifo_size;
> int32_t rx_fifo_used;
> int32_t rx_fifo_head;
> @@ -332,7 +332,9 @@ static const VMStateDescription vmstate_lan9118 = {
> VMSTATE_INT32(rx_status_fifo_size, lan9118_state),
> VMSTATE_INT32(rx_status_fifo_used, lan9118_state),
> VMSTATE_INT32(rx_status_fifo_head, lan9118_state),
> - VMSTATE_UINT32_ARRAY(rx_status_fifo, lan9118_state, 896),
> + VMSTATE_UINT32_ARRAY(rx_status_fifo, lan9118_state,
> + RX_STATUS_FIFO_BYTES / 4),
> + VMSTATE_UNUSED(896 * 4 - RX_STATUS_FIFO_BYTES),
> VMSTATE_INT32(rx_fifo_size, lan9118_state),
> VMSTATE_INT32(rx_fifo_used, lan9118_state),
> VMSTATE_INT32(rx_fifo_head, lan9118_state),
Ideally in the state struct we should have the arrays be the
size of the largest possible FIFO, not the size of the
default-out-of-reset FIFO, to leave the way open for making the FIFO
size be runtime configurable in future if we want that.
thanks
-- PMM
- [PATCH-for-9.1 v2 07/11] hw/net/lan9118: Rename tx_fifo_size -> tx_fifo_bytes, (continued)
- [PATCH-for-9.1 v2 07/11] hw/net/lan9118: Rename tx_fifo_size -> tx_fifo_bytes, Philippe Mathieu-Daudé, 2024/04/09
- [PATCH-for-9.1 v2 11/11] hw/net/lan9118: Rename rx_fifo_size -> rx_fifo_wordcount, Philippe Mathieu-Daudé, 2024/04/09
- [PATCH-for-9.1 v2 05/11] hw/net/lan9118: Add definitions for FIFO allocated sizes, Philippe Mathieu-Daudé, 2024/04/09
- [PATCH-for-9.1 v2 09/11] hw/net/lan9118: Rename rx_status_fifo_size -> rx_status_fifo_wordcount, Philippe Mathieu-Daudé, 2024/04/09
- [PATCH-for-9.1 v2 10/11] hw/net/lan9118: Use RX_DATA_FIFO_BYTES definition, Philippe Mathieu-Daudé, 2024/04/09
- [PATCH-for-9.1 v2 08/11] hw/net/lan9118: Use RX_STATUS_FIFO_BYTES definition, Philippe Mathieu-Daudé, 2024/04/09
- Re: [PATCH-for-9.1 v2 08/11] hw/net/lan9118: Use RX_STATUS_FIFO_BYTES definition,
Peter Maydell <=
- Re: [PATCH-for-9.0 v2 00/11] hw/net/lan9118: Fix overflow in TX FIFO, Philippe Mathieu-Daudé, 2024/04/09