Re: [PATCH 1/3] add a new option "testonly"

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

 



On 01/15/14 at 03:16pm, Vivek Goyal wrote:
> Bao,
> 
> This is harald's call but personally to me this testonly options look
> little odd.

Yeah, it does look odd. But I don't want this to affect the existed test
cases. 

> 
> I am looking at old discussion here.
> 
> http://thread.gmane.org/gmane.linux.kernel.initramfs/3370
> 
> So harald's suggestion of using systemd looks fine. It is just a matter of
> figuring out who should load watchdog modules in initramfs.
> 
> I personally feel it should be watchdog module but if harald does not
> like it, we can atleast for the time being keep that logic in kdump
> module. But trying to create this testonly mode feels odd to me.

Yeah, agree. For the time being, only iTCO_wdt need load module and
satisfacory. However, there are many kinds of watchdog with different
implementation. Finally, a independent dracut module may be needed, it
can be put into dracut as a independent folder, e.g 06wdt which is
different then 04watchdog. 

> 
> Harald, can you please let us know your preference on this. Modify
> 04watchdog module to include relevant drivers in hostonly mode or
> push this logic into kdump module for now.
> 
> Thanks
> Vivek
> 
> On Wed, Jan 15, 2014 at 08:53:48PM +0800, Baoquan He wrote:
--
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