RE: Is ib_mtu iboe_get_mtu() slightly off?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> From: linux-rdma-owner@xxxxxxxxxxxxxxx [mailto:linux-rdma-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Roland Dreier
> Sent: Wednesday, September 20, 2017 7:39 PM
> To: linux-rdma@xxxxxxxxxxxxxxx
> Subject: Is ib_mtu iboe_get_mtu() slightly off?
> 
> We have:
> 
> static inline enum ib_mtu iboe_get_mtu(int mtu) {
>         /*
>          * reduce IB headers from effective IBoE MTU. 28 stands for
>          * atomic header which is the biggest possible header after BTH
>          */
>         mtu = mtu - IB_GRH_BYTES - IB_BTH_BYTES - 28;
>        ...
> 
> 28 bytes is the size of the AtomicETH header.  But couldn't we have a packet
> with both AtomicETH and XRCETH (4 more bytes)?

XRCETH + AtomicETH is possible.
Additionally I guess 4 bytes of ICRC is missing too in calculation.
��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux