linux-kernel - Re: [PATCH 0/4] smpboot/watchdog: Fixes and cleanups

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 9 Jul 2015 16:59:07 -0400
From:	Chris Metcalf <cmetcalf@...hip.com>
To:	Frederic Weisbecker <fweisbec@...il.com>,
	LKML <linux-kernel@...r.kernel.org>
CC:	Peter Zijlstra <peterz@...radead.org>,
	Thomas Gleixner <tglx@...utronix.de>,
	Don Zickus <dzickus@...hat.com>,
	Ulrich Obergfell <uobergfe@...hat.com>,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH 0/4] smpboot/watchdog: Fixes and cleanups

On 07/09/2015 11:16 AM, Frederic Weisbecker wrote:
> Hi,
>
> The 2nd patch should fix the strange bug we've seen with Chris.
>
> git://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git
> 	core/watchdog
>
> HEAD: 5e36d90b3ccb0023ad54ac7a2381c132c6b12280
>
> Thanks,
> 	Frederic
> ---
>
> Frederic Weisbecker (4):
>        smpboot: Fix memory leak on error handling
>        smpboot: Make cleanup to mirror setup
>        smpboot: Allow to pass the cpumask on per-cpu thread registration
>        watchdog: Simplify housekeeping affinity with the appropriate mask
>
>
>   include/linux/smpboot.h | 11 ++++++++++-
>   kernel/smpboot.c        | 22 +++++++++-------------
>   kernel/watchdog.c       | 15 +++++----------
>   3 files changed, 24 insertions(+), 24 deletions(-)

You can add my

Reviewed-by: Chris Metcalf <cmetcalf@...hip.com>

to all four.  There are a couple of typos in the commit message for
patch 2/4: "ressources" and "lets simply" (should be "let's simply").

Glad you figured out what the bug was.  Curious that in your
environment you got a panic rather than an infinite loop waiting
for a timer subsystem pointer to go non-NULL.

-- 
Chris Metcalf, EZChip Semiconductor
http://www.ezchip.com

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/