From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Dumazet Subject: Re: [Bugme-new] [Bug 33502] New: Caught 64-bit read from uninitialized memory in __alloc_skb Date: Tue, 10 May 2011 13:52:34 +0200 Message-ID: <1305028354.2614.25.camel@edumazet-laptop> References: <1303181466.4152.39.camel@edumazet-laptop> <1303182557.4152.48.camel@edumazet-laptop> <1303183217.4152.49.camel@edumazet-laptop> <1303244270.2756.3.camel@edumazet-laptop> <4DAE7579.3020400@cs.helsinki.fi> <1303279470.2756.17.camel@edumazet-laptop> <1303285519.4dae8f0fdf9b1@imp.free.fr> <4DAE901C.2090809@cs.helsinki.fi> <1303286998.3186.18.camel@edumazet-laptop> <1303290464.3186.32.camel@edumazet-laptop> <1303293765.3186.74.camel@edumazet-laptop> <1303309591.3186.84.camel@edumazet-laptop> <1303311687.3186.100.camel@edumazet-laptop> <1305016988.2614.6.camel@edumazet-laptop> <4DC90D7D.9030808@cs.helsinki.fi> <1305022632.2614.18.camel@edumazet-laptop> <4DC91137.4030109@cs.helsinki.fi> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Christoph Lameter , casteyde.christian@free.fr, Andrew Morton , netdev@vger.kernel.org, bugzilla-daemon@bugzilla.kernel.org, bugme-daemon@bugzilla.kernel.org, Vegard Nossum To: Pekka Enberg Return-path: Received: from mail-ww0-f44.google.com ([74.125.82.44]:33722 "EHLO mail-ww0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754434Ab1EJLwi (ORCPT ); Tue, 10 May 2011 07:52:38 -0400 Received: by wwa36 with SMTP id 36so6781974wwa.1 for ; Tue, 10 May 2011 04:52:37 -0700 (PDT) In-Reply-To: <4DC91137.4030109@cs.helsinki.fi> Sender: netdev-owner@vger.kernel.org List-ID: Le mardi 10 mai 2011 =C3=A0 13:19 +0300, Pekka Enberg a =C3=A9crit : > On 5/10/11 1:17 PM, Eric Dumazet wrote: > > Le mardi 10 mai 2011 =C3=A0 13:03 +0300, Pekka Enberg a =C3=A9crit = : > > > >> Can't we fix the issue by putting kmemcheck_mark_initialized() to > >> set_freepointer()? > > > > This would solve kmemcheck problem, not DEBUG_PAGEALLOC >=20 > Oh, right. Christoph? We need to support DEBUG_PAGEALLOC with SLUB. Maybe have a special version of "irqsafe_cpu_cmpxchg_double" and make i= t use something like : local_irq_save(flags); if (probe_kernel_address(...)) { res =3D false; } else { res =3D unsafe_cpu_cmpxchg_double(...) } local_irq_restore(flags); Just the idea, I dont have time to cook a patch.