On Mar 10, 2011, at 11:07 AM, Mark Brown wrote: > On Thu, Mar 10, 2011 at 10:59:24AM -0500, Chuck Lever wrote: >> On Mar 10, 2011, at 10:43 AM, Mark Brown wrote: > >>> Commmit 972903 (NFS: NFSROOT should default to "proto=udp") added a >>> default NFS mount option using the existing in-kernel code to parse >>> mount options out of the command line. This exposed a bug in the > >> Looks like a real bug. > > Well, it's stopped me booting any of my systems with -next for a couple > of days :) Thanks for your testing. >> We might be better off teaching root_nfs_cat() to deal with the extra >> comma as necessary right when we concatenate the options strings. >> That would simplify it's callers. > > The thought did cross my mind, yes. That sounds like you'd like me to volunteer to do that. :-) -- Chuck Lever chuck[dot]lever[at]oracle[dot]com -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html