linux-kernel - [PATCHSET] stop_machine: implement stop_machine_from_offline

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-next>] [day] [month] [year] [list]

Message-Id: <1308071218-5912-1-git-send-email-tj@kernel.org>
Date:	Tue, 14 Jun 2011 19:06:55 +0200
From:	Tejun Heo <tj@...nel.org>
To:	x86@...nel.org, mingo@...e.hu, akpm@...ux-foundation.org,
	torvalds@...ux-foundation.org, suresh.b.siddha@...el.com,
	a.p.zijlstra@...llo.nl, linux-kernel@...r.kernel.org
Subject: [PATCHSET] stop_machine: implement stop_machine_from_offline_cpu()

Hello,

This three patch series implements stop_machine_from_offline_cpu(),
which is stop_machine() which can be used from a CPU which isn't yet
online.

This is for mtrr which needs stop_machine while bringing up a CPU.  It
currently implements its own stop_machine directly using
stop_one_cpu().  Duplicating such core functionality is wasteful and
fragile.  e.g. mtrr's implementation currently can deadlock against
generic stop_machine().

This patchset is born from Suresh's attempt at implementing similar
feature into [__]stop_machine()[1], which IMHO is a bit too subtle.
Given the peculiarity of the feature, I think it's better to have
dedicated function with an ugly name.  Also, this way, the
implementation is very well isolated from the rest of stop_cpus
machinery and much simpler.

This patchset contains the following three patches.

  0001-stop_machine-kill-__stop_machine.patch
  0002-stop_machine-reorganize-stop_cpus-implementation.patch
  0003-stop_machine-implement-stop_machine_from_offline_cpu.patch

0001 is a cleanup patch which can go in regardless of the rest (with
ack from x86 people of course).  0002 is pure code reorganization.
0003 implements the feature.

Suresh, how does this look to you?  Can you please put the mtrr
changes on top of this and see how it works?

 arch/x86/kernel/alternative.c |    5 -
 include/linux/stop_machine.h  |   42 +++------------
 kernel/cpu.c                  |    2 
 kernel/stop_machine.c         |  117 +++++++++++++++++++++++++++++++++++-------
 4 files changed, 114 insertions(+), 52 deletions(-)

Thanks.

--
tejun

[1] http://thread.gmane.org/gmane.linux.kernel/1154383/focus=1154384
--
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/