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! -- 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