> -----Original Message----- > From: Or Gerlitz [mailto:gerlitz.or@xxxxxxxxx] > Sent: Tuesday, November 14, 2017 4:21 PM > To: Parav Pandit <parav@xxxxxxxxxxxx> > Cc: Doug Ledford <dledford@xxxxxxxxxx>; linux-rdma@xxxxxxxxxxxxxxx; Hefty, > Sean <sean.hefty@xxxxxxxxx>; Leon Romanovsky <leonro@xxxxxxxxxxxx> > Subject: Re: [PATCH rdma-next 28/31] IB/cm: Fix avoid sleep while spin lock is > held > > On Wed, Nov 15, 2017 at 12:16 AM, Parav Pandit <parav@xxxxxxxxxxxx> > wrote: > >> From: Or Gerlitz [mailto:gerlitz.or@xxxxxxxxx] > >> Subject: Re: [PATCH rdma-next 28/31] IB/cm: Fix avoid sleep while > >> spin lock is > >>> From: Parav Pandit <parav@xxxxxxxxxxxx> > > >>> In case of LAP are used for RoCE > > >> AFAIK APM is not supported with RoCE, isn't that? if yes, on what > >> HW/RoCE mode and how do you test that? > > > This particular patch in this series is not tested on RoCE. > > APM is not supported but injecting such frame in network is not impossible. > > We don’t have particular checks to reject such frame for RoCE. > > So I prefer to fix the issue before getting such hang/oops report when issue is > evident in code. > > mentioning testing, in this series you did bunch of fixes for the IB core, CM and > CMA. > > Would be good to add a section in the cover letter mentioning under which type > of testing the modified code has gone through. It's been tested for RoCEv1, v2, IPoIB with ConnectX4, ConnectX3 HCA with perftest, rping, krping tools. In few individual patch, it's been mentioned how is this tested. > > Seems that you did a hell of hardening to the IB core, cool!! Thanks Or. ��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f
