Re: [PATCH 2/2] Remove unnecessary assignment of EthernetHeader

[Paul Durrant <xadimgnik@xxxxxxxxx>]

On 05/03/2024 15:33, Owen Smith wrote:
> BaseVa and EthernetHeader are not used after this point in the function,
> remove the unnecessary assignment
>
> Signed-off-by: Owen Smith <owen.smith@xxxxxxxxx>
Reviewed-by: Paul Durrant <paul@xxxxxxx>

> ---
>   src/xenvif/receiver.c | 4 ----
>   1 file changed, 4 deletions(-)
>
> diff --git a/src/xenvif/receiver.c b/src/xenvif/receiver.c
> index 37e2c60..df5ed6f 100644
> --- a/src/xenvif/receiver.c
> +++ b/src/xenvif/receiver.c
> @@ -469,10 +469,6 @@ ReceiverRingProcessTag(
>   
>       Info->Length -= sizeof (ETHERNET_TAG);
>   
> -    BaseVa += Packet->Offset;
> -
> -    EthernetHeader = (PETHERNET_HEADER)(BaseVa + Info->EthernetHeader.Offset);
> -
>       ASSERT3U(PayloadLength, ==, Packet->Length - Info->Length);
>   }
>