> -----Original Message----- > From: Amrani, Ram [mailto:Ram.Amrani@xxxxxxxxxx] > Sent: Tuesday, October 10, 2017 5:30 AM > To: Leon Romanovsky <leon@xxxxxxxxxx>; Doug Ledford > <dledford@xxxxxxxxxx>; Parav Pandit <parav@xxxxxxxxxxxx>; Roland Dreier > <roland@xxxxxxxxxxxxxxx>; linux-rdma@xxxxxxxxxxxxxxx > Subject: RE: Is ib_mtu iboe_get_mtu() slightly off? > > > Fixes: 3c86aa70bf67 ("RDMA/cm: Add RDMA CM support for IBoE devices") > > Signed-off-by: Parav Pandit <parav@xxxxxxxxxxxx> > > Reviewed-by: Daniel Jurgens <danielj@xxxxxxxxxxxx> > > Reported-by: Roland Dreier <roland@xxxxxxxxxxxxxxx> > > Signed-off-by: Leon Romanovsky <leon@xxxxxxxxxx> > > Hi Parav, All, > Shouldn't the overhead calculation take into account RoCEv2 too, i.e. include > the UDP header length? > Yes. I will roll v1 with inclusion of UDP header as well. Now that we have RoCEv2 GIDs as default, it make sense to always include UDP header in calculation. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
