[ofw] [PATCH 1/3] IBAL (checksum): restoring vendor_specific field
Fab Tillier
ftillier at windows.microsoft.com
Thu Sep 4 11:18:20 PDT 2008
Hi Xalex,
Thank you for doing this.
> Index: D:/Windows/MLNX_WINOF_CSUM_ARBEL/inc/iba/ib_types.h
> =================================================================== ---
> @@ -10670,6 +10665,7 @@
> ib_net16_t remote_lid;
> uint8_t remote_sl;
> uint8_t path_bits;
> + uint8_t csum_ok;
>
> } ud;
Can you reserve the csum_ok bits in the recv_opt field rather than adding an extra field? This opens up the possibility in the future to have checksum offload for RC connections without needing to change the interfaces further. It also removes the need for drivers to clear the field if checksum isn't supported (since the unused bits in the recv_opt field should be cleared).
-Fab
More information about the ofw
mailing list