Re: [PATCH 1/2 v2] split nfsroot rpc start code

[Dave Young <dyoung-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

On 02/01/2012 04:00 PM, Dave Young wrote:

> On 02/01/2012 03:56 PM, Cong Wang wrote:
> 
>> On 02/01/2012 03:56 PM, Dave Young wrote:
>>> On 02/01/2012 03:48 PM, Cong Wang wrote:
>>>> On Wed, 01 Feb 2012 at 02:00 GMT, Dave Young<dyoung-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>  wrote:
>>>>> Index: dracut/modules.d/95nfs/module-setup.sh
>>>>> ===================================================================
>>>>> --- dracut.orig/modules.d/95nfs/module-setup.sh
>>>>> +++ dracut/modules.d/95nfs/module-setup.sh
>>>>> @@ -57,6 +57,7 @@ install() {
>>>>>       dracut_install $(for _i in $(ls {/usr,}$libdir/libnss*.so
>>>>> 2>/dev/null); do echo $_i;done | egrep "$_nsslibs")
>>>>>
>>>>>       inst_hook cmdline 90 "$moddir/parse-nfsroot.sh"
>>>>> +    inst_hook cmdline 91 "$moddir/nfs-start-rpc.sh"
>>>>
>>>> I am sure this is wrong. You need pre-pivot.
>>>
>>>
>>> I want to start them as early as we can, I always think about user use
>>> dracut as a minimal recovery system, they need do something manually.
>>>
>>
>> Then install it as pre-pivot 00.
> 
> 
> What if I use rdbreak=point which is before pre-pivot?
> 
>>
>> Do one thing at one time, I know this patch now is for kdump. If you
>> really need a minimal recovery system, do that later. :)


Generally it's right, but for this issue they are not conflict. I'd like
to think it as a general feature and do not assume it's only for kdump.

-- 
Thanks
Dave