> -----Original Message----- > From: linux-rdma-owner@xxxxxxxxxxxxxxx [mailto:linux-rdma- > owner@xxxxxxxxxxxxxxx] On Behalf Of Chuck Lever > Sent: Thursday, July 31, 2014 12:09 AM > To: Devesh Sharma > Cc: Steve Wise; Shirley Ma; Hefty, Sean; Roland Dreier; linux- > rdma@xxxxxxxxxxxxxxx > Subject: Re: [for-next 1/2] xprtrdma: take reference of rdma provider > module > > > On Jul 22, 2014, at 1:06 AM, Devesh Sharma <Devesh.Sharma@xxxxxxxxxx> > wrote: > > >> -----Original Message----- > >> From: Chuck Lever [mailto:chuck.lever@xxxxxxxxxx] > >> Sent: Monday, July 21, 2014 11:01 PM > >> To: Devesh Sharma > >> Cc: Steve Wise; Shirley Ma; Hefty, Sean; Roland Dreier; linux- > >> rdma@xxxxxxxxxxxxxxx > >> Subject: Re: [for-next 1/2] xprtrdma: take reference of rdma provider > >> module > >> > >> > >> On Jul 21, 2014, at 1:07 PM, Devesh Sharma > <Devesh.Sharma@xxxxxxxxxx> > >> wrote: > >> > >>>>>>> Until that support is in place, obviously I would prefer that > >>>>>>> the removal of the underlying driver be prevented while there > >>>>>>> are NFS mounts in place. I think that's what NFS users have come to > expect. > >>>>>>> > >>>>>>> In other words, don't allow device removal until we have support > >>>>>>> for device insertion :-) > >>> > >>> This needs a fresh series of patches? > >> > >> "do not allow removal" would likely take an approach similar to what > >> you originally posted, unless someone has an idea how to use a > >> CM_EVENT to make this work, or there is an objection from the kernel > RDMA folks. > > > > Okay, I will re-work the patch if need be. > > Devesh, if there isn't one already, could you file an upstream bug at > > http://bugzilla.linux-nfs.org > > that documents the shutdown hang and perhaps summarizes this thread? > Thanks! A bug has been filed https://bugzilla.linux-nfs.org/show_bug.cgi?id=266 > > -- > Chuck Lever > chuck[dot]lever[at]oracle[dot]com > > > > -- > 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 -- 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