Messages in this thread | | | Date | Fri, 26 Jul 2013 22:07:34 +0200 | From | Ingo Molnar <> | Subject | Re: [PATCH 1/2] watchdog: update watchdog attributes atomically |
| |
* Don Zickus <dzickus@redhat.com> wrote:
> On Fri, Jul 26, 2013 at 02:49:34PM +0200, Michal Hocko wrote: > > proc_dowatchdog doesn't synchronize multiple callers which > > might lead to confusion when two parallel callers might confuse > > watchdog_enable_all_cpus resp. watchdog_disable_all_cpus (e.g. watchdog > > gets enabled even if watchdog_thresh was set to 0 already). > > > > This patch adds a local mutex which synchronizes callers to the sysctl > > handler. > > > > Signed-off-by: Michal Hocko <mhocko@suse.cz> > > Acked-by: Don Zickus <dzickus@redhat.com>
Ack from you for both patches?
Thanks,
Ingo
| |