[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAMuHMdUugcEPjN+1KoLocpe0GsoXoi5qeXMXvfQSbhm5XDjv7Q@mail.gmail.com>
Date: Mon, 11 Dec 2017 09:08:15 +0100
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Linus Torvalds <torvalds@...ux-foundation.org>
Cc: Geert Uytterhoeven <geert+renesas@...der.be>,
Christoph Lameter <cl@...ux.com>,
Pekka Enberg <penberg@...nel.org>,
David Rientjes <rientjes@...gle.com>,
Joonsoo Kim <iamjoonsoo.kim@....com>,
Andrew Morton <akpm@...ux-foundation.org>,
"Tobin C . Harding" <me@...in.cc>, linux-mm <linux-mm@...ck.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] mm/slab: Do not hash pointers when debugging slab
Hi Linus,
On Sun, Dec 10, 2017 at 9:45 PM, Linus Torvalds
<torvalds@...ux-foundation.org> wrote:
> On Thu, Dec 7, 2017 at 2:17 AM, Geert Uytterhoeven
> <geert+renesas@...der.be> wrote:
>> - pr_err("Slab corruption (%s): %s start=%p, len=%d\n",
>> + pr_err("Slab corruption (%s): %s start=%px, len=%d\n",
>> print_tainted(), cachep->name,
>> realobj, size);
>
> and here, is the pointer actually interesting, or should we just give
> the offset to the allocation?
The pointer may help to identify e.g. an empty list_head in the written data.
> But if the pointer is interesting, then ack.
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
Powered by blists - more mailing lists