From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leon Romanovsky Subject: Re: [PULL REQUEST] Please pull rdma.git Date: Fri, 26 Aug 2016 22:19:51 +0300 Message-ID: <20160826191951.GE594@leon.nu> References: <20160826144415.GC594@leon.nu> <3aee5577-9600-db32-db7f-4fb39afdc429@redhat.com> <20160826171638.GD594@leon.nu> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="YkJPYEFdoxh/AXLE" Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Doug Ledford Cc: "Torvalds, Linus" , linux-rdma List-Id: linux-rdma@vger.kernel.org --YkJPYEFdoxh/AXLE Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Aug 26, 2016 at 01:53:03PM -0400, Doug Ledford wrote: > On 8/26/2016 1:16 PM, Leon Romanovsky wrote: > > On Fri, Aug 26, 2016 at 12:12:07PM -0400, Doug Ledford wrote: > >> On 8/26/2016 10:44 AM, Leon Romanovsky wrote: > >>> On Thu, Aug 25, 2016 at 03:29:12PM -0400, Doug Ledford wrote: > >>> > >>> Hi Doug, > >>> > >>> These two patches were supposed to be carried by us [1] and I explicitly > >>> said that. Especially, the last patch in this series is wrong. Please > >>> revert it. The proper patch is [2] and it was supposed to be sent right > >>> after our shared code. > >>> > >>>> Yuval Shaia (2): > >>>> IB/mlx4: Make function use_tunnel_data return void > >>>> IB/mlx4: Return EAGAIN for any error in mlx4_ib_poll_one > >>> > >>> [1] https://www.spinics.net/lists/linux-rdma/msg38580.html > >>> [2] > >>> https://git.kernel.org/cgit/linux/kernel/git/leon/linux-rdma.git/commit/?h=topic/mlx-fixes&id=d5a1c56c3a49db6369b36aa04fbeabc7653ae107 > >>> > >> > >> I saw your request to carry it, but then never heard anything again from > >> you. The fix that the original patch implements is still valid and was > >> something I wanted to get into 4.8, where as your patch is being held > >> for 4.9 and goes down a different path to solving the issue. IMO, the > >> patch from Yuval is fine for 4.8, and if you want to apply yours to 4.9, > >> that's fine too. I don't see a need to revert the existing patch. From > >> what I can tell, the existing patch will work fine and do what Yuval > >> intended, it just won't do what you intend to do in 4.9. Please correct > >> me if I'm wrong. > > > > By our HW design and SW implementation poll_cq never fails and returns > > errors, so all these prints are to catch ULP bugs. In case of such bug, Yuval's > > patch will cause to reentry (EAGAIN) and kprints storm again and again. > > It is undesired and misleading behaviour. > > > > We targeted our patch to 4.9, because it is not actual fix, but help to > > ULP developers and there is no real need to hurry up. > > > > Will it be acceptable by you, if I revert Yuval's patch before sending > > our version for 4.9? > > Just post your version now. I have another pull request for 4.8-rc, > I'll merge in your version into the next pull request. I will do it immediately after I'll return to my office (Sunday). Thanks > > > -- > Doug Ledford > GPG Key ID: 0E572FDD > --YkJPYEFdoxh/AXLE Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJXwJZXAAoJEORje4g2clinYuAQAMzbhF5OjGJX2+O3RenSdnuJ 1q6JsRv5FmIZzsMijPgB7orSLq2SX/nA0GyX3Tzt9wGTp3Ou52A9nVcODddHD/YN dKjLTpw6rFV1mk0Klc6vO1/iVd+qNqMzb5y1KwWNRvTWqM6Aa6UQ0EOVUz+DrnJ3 rG/0V0Gtq7p4RSKIX7Pv6c5PPjBJwN+AsYH9bN33psg+6eyybBKXkV5jitMETs2l 9GXnKlbZebwft8dpeTQl3as7D1Yr1l4/RJbqzxvAwikhGaFzaoWYsd6pi56Jlyn3 jwTjJIhPZQkV2+dHWb5nDUppuAx+Vf0ChgS5dn830xDZttSTyi0b4wRe/Z3tYvEE bX8AzoZt69zPMT2gl7yskw/hUmuN1uEwHBeRwV7CbTilQRpiGSOy1b42PNUGJ+cw He/gbIqlzADpzvkjm/w14uDDW/2NoqVUCdG5qSsLj5bcLyB+dy9dlYk6bOkMnnQI Ussz9d6kY0gejq1gsWZAOyLHNCVGcOEuZPCCFzMFL2jwObSE3dyYiG9SyqZn6sxS NW8xt3HoPKZsWH3P6Hr8pGlH2Slz13DZpiv0yvvTlPrYcu5JnJhuJ3YbnUqGl1BS v2WXlzkwG2b+glo3KnTT+6E1Qz8+w5sqjzXvw7HksQSPfFrogPgEJYVU1AMVDraz mwmoUnG0a1M20VqKFUpd =g92Z -----END PGP SIGNATURE----- --YkJPYEFdoxh/AXLE-- -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html