RE: [PATCH net-next] net: mana: Add RX fencing

From: Haiyang Zhang
Date: Thu Dec 16 2021 - 11:44:53 EST




> -----Original Message-----
> From: Dexuan Cui <decui@xxxxxxxxxxxxx>
> Sent: Wednesday, December 15, 2021 7:18 PM
> To: davem@xxxxxxxxxxxxx; kuba@xxxxxxxxxx; gustavoars@xxxxxxxxxx; Haiyang Zhang
> <haiyangz@xxxxxxxxxxxxx>; netdev@xxxxxxxxxxxxxxx
> Cc: KY Srinivasan <kys@xxxxxxxxxxxxx>; stephen@xxxxxxxxxxxxxxxxxx; wei.liu@xxxxxxxxxx;
> linux-kernel@xxxxxxxxxxxxxxx; linux-hyperv@xxxxxxxxxxxxxxx; Shachar Raindel
> <shacharr@xxxxxxxxxxxxx>; Paul Rosswurm <paulros@xxxxxxxxxxxxx>; olaf@xxxxxxxxx; vkuznets
> <vkuznets@xxxxxxxxxx>; Dexuan Cui <decui@xxxxxxxxxxxxx>
> Subject: [PATCH net-next] net: mana: Add RX fencing
>
> RX fencing allows the driver to know that any prior change to the RQs has
> finished, e.g. when the RQs are disabled/enabled or the hashkey/indirection
> table are changed, RX fencing is required.
>
> Remove the previous workaround "ssleep(1)" and add the real support for
> RX fencing as the PF driver supports the MANA_FENCE_RQ request now (any
> old PF driver not supporting the request won't be used in production).
>
> Signed-off-by: Dexuan Cui <decui@xxxxxxxxxxxxx>
> ---
> drivers/net/ethernet/microsoft/mana/mana.h | 2 +
> drivers/net/ethernet/microsoft/mana/mana_en.c | 69 +++++++++++++++++--
> 2 files changed, 66 insertions(+), 5 deletions(-)
>

Reviewed-by: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx>

Thanks!