From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:57584) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJsy9-0007D3-Jy for qemu-devel@nongnu.org; Thu, 04 Oct 2012 17:28:34 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TJsy8-0003FZ-66 for qemu-devel@nongnu.org; Thu, 04 Oct 2012 17:28:33 -0400 Received: from mail-pb0-f45.google.com ([209.85.160.45]:47436) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJsy8-0003FG-0V for qemu-devel@nongnu.org; Thu, 04 Oct 2012 17:28:32 -0400 Received: by mail-pb0-f45.google.com with SMTP id rp2so1001628pbb.4 for ; Thu, 04 Oct 2012 14:28:31 -0700 (PDT) From: Greg Kroah-Hartman Date: Thu, 4 Oct 2012 14:26:52 -0700 Message-Id: <20121004210602.236827150@linuxfoundation.org> In-Reply-To: <20121004210558.383865383@linuxfoundation.org> References: <20121004210558.383865383@linuxfoundation.org> Subject: [Qemu-devel] [ 30/33] x86/alternatives: Fix p6 nops on non-modular kernels List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Greg Kroah-Hartman , Marcelo Tosatti , Michael Tokarev , Borislav Petkov , Tomas Racek , qemu-devel@nongnu.org, Avi Kivity , Anthony Liguori , "H. Peter Anvin" , Ben Jencks , Ingo Molnar , alan@lxorguk.ukuu.org.uk, Alan Cox 3.0-stable review patch. If anyone has any objections, please let me know. ------------------ From: Avi Kivity commit cb09cad44f07044d9810f18f6f9a6a6f3771f979 upstream. Probably a leftover from the early days of self-patching, p6nops are marked __initconst_or_module, which causes them to be discarded in a non-modular kernel. If something later triggers patching, it will overwrite kernel code with garbage. Reported-by: Tomas Racek Signed-off-by: Avi Kivity Cc: Michael Tokarev Cc: Borislav Petkov Cc: Marcelo Tosatti Cc: qemu-devel@nongnu.org Cc: Anthony Liguori Cc: H. Peter Anvin Cc: Alan Cox Cc: Alan Cox Link: http://lkml.kernel.org/r/5034AE84.90708@redhat.com Signed-off-by: Ingo Molnar Cc: Ben Jencks Signed-off-by: Greg Kroah-Hartman --- arch/x86/kernel/alternative.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/arch/x86/kernel/alternative.c +++ b/arch/x86/kernel/alternative.c @@ -161,7 +161,7 @@ static const unsigned char * const k7_no #endif #ifdef P6_NOP1 -static const unsigned char __initconst_or_module p6nops[] = +static const unsigned char p6nops[] = { P6_NOP1, P6_NOP2,