Re: [PATCH] deal with nfs in fstab-sys

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

 



On 01/30/2012 05:27 PM, Cong Wang wrote:

> On 01/30/2012 05:20 PM, Dave Young wrote:
>> On 01/30/2012 05:12 PM, Cong Wang wrote:
>>
>>> On 01/30/2012 05:02 PM, Dave Young wrote:
>>>> On 01/30/2012 04:43 PM, Cong Wang wrote:
>>>>
>>>>> On 01/30/2012 04:41 PM, Dave Young wrote:
>>>>>> On 01/30/2012 04:20 PM, Cong Wang wrote:
>>>>>>
>>>>>>> On Mon, 30 Jan 2012 at 06:44 GMT, Dave Young<dyoung@xxxxxxxxxx>
>>>>>>> wrote:
>>>>>>>> If we need to mount nonroot nfs in fstab we also need to launch rpc
>>>>>>>> processes
>>>>>>>> Add nfs-start-rpc code, also do not check and fsck $_dev in this
>>>>>>>> case
>>>>>>>
>>>>>>> Isn't it nfs module's responsibility to call nfs-start-rpc code?
>>>>>>> Why do you put it in fs-systab?
>>>>>>
>>>>>>
>>>>>> nfs know nothing about fstab mount, it only deal with nfsroot
>>>>>> thing. So
>>>>>> I have to do this in fstab-sys
>>>>>
>>>>> Sure, but nfs module knows it needs to start rpc before mounting NFS.
>>>>>
>>>>> So, what's the problem if we call nfs-start-rpc script in pre-pivot
>>>>> hook
>>>>> of nfs module (before mount-sys.sh, of course)?
>>>>
>>>>
>>>> Actually nfs-start-rpc need to know the fs is nfs or nfs4, If add it as
>>>> a hook, how does it accept the argument?
>>>
>>> I think you can start rpc daemons unconditionally, no matter if it is
>>> nfs4 or nfs.
>>
>>
>> This will means once nfs module is installed in initramfs, the rpc
>> daemons will be started even if no nfs mount options in fstab or nfsroot
> 
> Sure, but they will be killed soon, by nfsroot-cleanup.sh. :)


Rethink about this, unconditionally start rpc daemons make more sense
than do it in fstab-sys. Will change in v2


-- 
Thanks
Dave
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux