Re: [PATCH] NFS: Fix handling of commmand line options for NFSROOT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux