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

[Vivek Goyal <vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

On Mon, Jan 20, 2014 at 10:28:48AM +0800, Baoquan He wrote:

[..]

> > 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. 

I think creating another watchdog module 06wdt does not make much sense.
Eiter this logic needs to be merged into existing watchdog module 04wdt
or we modify kdump module to load watchdog drivers in kdump initramfs
as a short term approach till we can convince harald that keeping this
logic in 04watchdog is a good idea.

Thanks
Vivek