All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pratyush Anand <panand-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Dave Young <dyoung-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	dzickus-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org,
	harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org
Subject: Re: [PATCH V2 dracut 0/2] Install kernel module for active watchdog
Date: Fri, 4 Mar 2016 09:04:43 +0530	[thread overview]
Message-ID: <20160304033443.GB7467@dhcppc3.redhat.com> (raw)
In-Reply-To: <20160304031633.GB3592-0VdLhd/A9Pl+NNSt+8eSiB/sF2h8X+2i0E9HWUfgJXw@public.gmane.org>

Hi Dave,

On 04/03/2016:11:16:33 AM, Dave Young wrote:
> Hi, Pratyush
> 
> On 03/02/16 at 05:36pm, Pratyush Anand wrote:
> > Changes since RFC:
> >  -- Added one more patch to take care that dracut does not add watchdog
> > hooks when systemd module is included. In that case systemd daemon will
> > manage watchdog for kicking.
> >  -- Removed --nowdt argument. User can use -o or --omit when wathdog module
> > is not needed. Thanks Harald for this suggestion.
> >  -- Now I also do not check for "hostonly" to add modules for active
> > watchdog. It seems reasonable to add kernel watchdog modules whenever,
> > dracut watchdog module has been added.
> 
> >From my understanding hostonly is a different mode comparing with the default
> one which being used for distribution provided initramfs. In the latter case
> all possible drivers should be included. But hostonly mode only adding the
> drivers being used in this boot. From the code logic we provided it will detect
> active wdt module and add the driver, the behavior is just "hostonly", no?

I think, I am agreeing to this.

So, should we also add no "hostonly" case where driver will be present
irrespective of whether wdt is active or not? May be we can leave it for now,
and can be added if someone needs it latter.

~Pratyush

  parent reply	other threads:[~2016-03-04  3:34 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-02 12:06 [PATCH V2 dracut 0/2] Install kernel module for active watchdog Pratyush Anand
     [not found] ` <cover.1456919929.git.panand-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-03-02 12:06   ` [PATCH V2 dracut 1/2] watchdog: Do not add hooks if systemd module is included Pratyush Anand
2016-03-02 12:06   ` [PATCH V2 dracut 2/2] watchdog: install module for active watchdog Pratyush Anand
     [not found]     ` <0d7adcfdf08f4eb753a0e65e7e65d5fe0ef22f70.1456919929.git.panand-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-03-02 15:04       ` Dracut GitHub Import Bot
2016-03-04  3:13       ` Dave Young
     [not found]         ` <20160304031334.GA3592-0VdLhd/A9Pl+NNSt+8eSiB/sF2h8X+2i0E9HWUfgJXw@public.gmane.org>
2016-03-04  4:15           ` Pratyush Anand
     [not found]             ` <20160304041550.GC7467-AB7H14ik7lQf7BdofF/totBPR1lH4CV8@public.gmane.org>
2016-03-04  4:53               ` Dave Young
2016-03-04  6:22               ` Pratyush Anand
2016-03-04  4:56       ` Dave Young
     [not found]         ` <20160304045610.GD3592-0VdLhd/A9Pl+NNSt+8eSiB/sF2h8X+2i0E9HWUfgJXw@public.gmane.org>
2016-03-04  5:09           ` Pratyush Anand
     [not found]             ` <20160304050914.GD7467-AB7H14ik7lQf7BdofF/totBPR1lH4CV8@public.gmane.org>
2016-03-04  7:30               ` Dave Young
     [not found]                 ` <20160304073007.GB2909-0VdLhd/A9Pl+NNSt+8eSiB/sF2h8X+2i0E9HWUfgJXw@public.gmane.org>
2016-03-04  7:44                   ` Pratyush Anand
     [not found]                     ` <20160304074422.GG7467-AB7H14ik7lQf7BdofF/totBPR1lH4CV8@public.gmane.org>
2016-03-04  8:18                       ` Dave Young
2016-03-07 13:49       ` Harald Hoyer
     [not found]         ` <56DD86D4.8040800-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-03-07 16:28           ` Pratyush Anand
     [not found]             ` <20160307162813.GC20953-AB7H14ik7lQf7BdofF/totBPR1lH4CV8@public.gmane.org>
2016-03-07 16:52               ` Harald Hoyer
     [not found]                 ` <56DDB1DF.7050500-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-03-08  2:15                   ` Pratyush Anand
2016-03-04  3:16   ` [PATCH V2 dracut 0/2] Install kernel " Dave Young
     [not found]     ` <20160304031633.GB3592-0VdLhd/A9Pl+NNSt+8eSiB/sF2h8X+2i0E9HWUfgJXw@public.gmane.org>
2016-03-04  3:34       ` Pratyush Anand [this message]
     [not found]         ` <20160304033443.GB7467-AB7H14ik7lQf7BdofF/totBPR1lH4CV8@public.gmane.org>
2016-03-04  7:26           ` Dave Young

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160304033443.GB7467@dhcppc3.redhat.com \
    --to=panand-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=dyoung-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=dzickus-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.