On Tue, Aug 01, 2017 at 05:10:23PM +0300, Leon Romanovsky wrote:
> On Tue, Aug 01, 2017 at 08:38:32AM -0500, Chien Tin Tung wrote:
> > On Tue, Aug 01, 2017 at 03:05:04PM +0300, Leon Romanovsky wrote:
> > > From: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> > >
> > > The commit cea05eadded0 ("IB/core: Add flow control to the portmapper netlink calls")
> > > changed netlink to be blocked for all RDMA clients. This workaround
> > > worked perfectly for portmapper, but is not correct for the whole
> > > NETLINK_RDMA family.
> >
> > Leon,
> >
> > I've already told you I'm opposed to the revert. There is a patch that will work
> > with your usage of ibnl_unicast() but you chose to abandon that discussion on June 29, 2017
> > (RDMA/core: Add wait/retry version of ibnl_unicast). Either you step up with good sound
> > technical evidence to convince me that patch won't work for you or you stop trying to
> > break existing functionality with this revert.
>
> Chien,
>
> You never explained us what exactly your original patch fixed and why it
> should be fixed in kernel and not in user space. I saw that our
> discussion wasn't useful and brought bad blood instead of good will,
> so I stepped out in waited for RDMA maintainer (Doug) step in.
Why do I need to explain the original patch? It was accepted many kernels
ago. Your questions on the original were based on false assumptions and
facts which I've proven over and over. You are right in that I do
not want to revisit those either. There is a patch that can solve
the problem you are facing but yet you insist on the revert. This is
very puzzling to me. You simply refuse to move this forward. That
is your choice but revert is a no go for me.
> I never heard anything from Doug on the matter, so proceeded and
> resubmitted it.
Fine. Your choice to resubmit and my choice to say no to the revert.
Chien
--
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
