Hi Chuk, Can this patch be lined up for merger. Looks like there are no oppositions to this change
-Regards Devesh > -----Original Message----- > From: [email protected] [mailto:linux-rdma- > [email protected]] On Behalf Of Devesh Sharma > Sent: Thursday, July 31, 2014 10:45 AM > To: Chuck Lever > Cc: Steve Wise; Shirley Ma; Hefty, Sean; Roland Dreier; linux- > [email protected] > Subject: RE: [for-next 1/2] xprtrdma: take reference of rdma provider > module > > > -----Original Message----- > > From: [email protected] [mailto:linux-rdma- > > [email protected]] 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- > > [email protected] > > Subject: Re: [for-next 1/2] xprtrdma: take reference of rdma provider > > module > > > > > > On Jul 22, 2014, at 1:06 AM, Devesh Sharma > <[email protected]> > > wrote: > > > > >> -----Original Message----- > > >> From: Chuck Lever [mailto:[email protected]] > > >> Sent: Monday, July 21, 2014 11:01 PM > > >> To: Devesh Sharma > > >> Cc: Steve Wise; Shirley Ma; Hefty, Sean; Roland Dreier; linux- > > >> [email protected] > > >> Subject: Re: [for-next 1/2] xprtrdma: take reference of rdma > > >> provider module > > >> > > >> > > >> On Jul 21, 2014, at 1:07 PM, Devesh Sharma > > <[email protected]> > > >> 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 [email protected] 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 [email protected] 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
