Re: [regression] nf_iterate(), BUG: unable to handle kernel NULL pointer dereference

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Herbert Xu <herbert@...>
Cc: Pekka Enberg <penberg@...>, Patrick McHardy <kaber@...>, Ingo Molnar <mingo@...>, David Miller <davem@...>, <w@...>, <davidn@...>, <torvalds@...>, <akpm@...>, <netdev@...>, <linux-kernel@...>, <stefanr@...>, <rjw@...>, <ilpo.jarvinen@...>, Dave Jones <davej@...>, Christoph Lameter <cl@...>
Date: Thursday, July 24, 2008 - 1:47 pm

On Thu, 2008-07-24 at 22:37 +0800, Herbert Xu wrote:

Let's try this again: did you know that ksize could fail depending on
kernel configuration? Most of us would answer no. That suggests the API
is bad. This ranks 12 on Rusty's spectrum of user-friendly APIs:

http://ozlabs.org/~rusty/ols-2003-keynote/img51.html


Ahh. I see what you're saying. You may be right about that, or you may
be wrong: I don't know which of the millions of mass market embedded
Linux devices out there are using it.

And of course I argue that SLOB did do the right thing, which was only
allowing ksize on kmalloced objects. It's an accident of implementation
that kmalloc and kmem_cache_alloc use the same underlying allocator. It
has not been true at all points in the past, it's not true for some
users in the present, and it may not be true for most users in the
future. Thus, it's a bad idea to try to use ksize on something that
wasn't kmalloced.

If you have an argument for reintroducing ksize based on an actual use
case, let's please move on to (or back to) it so that we can have some
substance to this discussion.

-- 
Mathematics is the supreme nostalgia of our time.

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[GIT]: Networking, David Miller, (Sun Jul 20, 1:44 pm)
Re: [GIT]: Networking, Ingo Molnar, (Mon Jul 21, 9:50 am)
Re: [GIT]: Networking, Stefan Richter, (Mon Jul 21, 10:15 am)
[crash] kernel BUG at net/core/dev.c:1328!, Ingo Molnar, (Mon Jul 21, 2:23 pm)
Re: [crash] kernel BUG at net/core/dev.c:1328!, Linus Torvalds, (Mon Jul 21, 2:35 pm)
Re: [crash] kernel BUG at net/core/dev.c:1328!, David Miller, (Mon Jul 21, 3:00 pm)
Re: [crash] kernel BUG at net/core/dev.c:1328!, Ingo Molnar, (Mon Jul 21, 3:44 pm)
Re: [crash] kernel BUG at net/core/dev.c:1328!, David Miller, (Mon Jul 21, 4:20 pm)
Re: [crash] kernel BUG at net/core/dev.c:1328!, Stefan Richter, (Mon Jul 21, 3:20 pm)
Re: [crash] kernel BUG at net/core/dev.c:1328!, David Miller, (Mon Jul 21, 4:11 pm)
Re: [crash] kernel BUG at net/core/dev.c:1328!, Stefan Richter, (Mon Jul 21, 5:26 pm)
Re: [crash] kernel BUG at net/core/dev.c:1328!, Ingo Molnar, (Mon Jul 21, 2:46 pm)
[TCP bug] stuck distcc connections in latest -git, Ingo Molnar, (Tue Jul 22, 7:21 am)
Re: [TCP bug] stuck distcc connections in latest -git, David Newall, (Tue Jul 22, 9:45 am)
Re: [TCP bug] stuck distcc connections in latest -git, Ingo Molnar, (Tue Jul 22, 9:57 am)
Re: [TCP bug] stuck distcc connections in latest -git, David Newall, (Tue Jul 22, 10:54 am)
Re: [TCP bug] stuck distcc connections in latest -git, Ingo Molnar, (Tue Jul 22, 11:34 am)
Re: [TCP bug] stuck distcc connections in latest -git, Willy Tarreau, (Tue Jul 22, 5:12 pm)
Re: [TCP bug] stuck distcc connections in latest -git, Ingo Molnar, (Wed Jul 23, 4:26 am)
Re: [regression] nf_iterate(), BUG: unable to handle kernel ..., Krzysztof Oledzki, (Thu Jul 24, 2:00 pm)
Re: [regression] nf_iterate(), BUG: unable to handle kernel ..., Matt Mackall, (Thu Jul 24, 1:47 pm)
Re: [crash] kernel BUG at net/core/dev.c:1328!, Ingo Molnar, (Mon Jul 21, 3:30 pm)
iwlwifi: fix build bug in "iwlwifi: fix LED stall", Ingo Molnar, (Mon Jul 21, 4:36 am)
RE: iwlwifi: fix build bug in "iwlwifi: fix LED stall", Winkler, Tomas, (Mon Jul 21, 6:02 am)
Re: iwlwifi: fix build bug in "iwlwifi: fix LED stall", Ingo Molnar, (Mon Jul 21, 6:53 am)
Re: [GIT]: Networking, Linus Torvalds, (Sun Jul 20, 8:54 pm)
Re: [GIT]: Networking, Linus Torvalds, (Sun Jul 20, 9:07 pm)
Re: [GIT]: Networking, David Miller, (Sun Jul 20, 9:17 pm)
Re: [GIT]: Networking, David Miller, (Sun Jul 20, 9:03 pm)
Re: [GIT]: Networking, Patrick McHardy, (Sun Jul 20, 9:20 pm)
Re: [GIT]: Networking, James Morris, (Mon Jul 21, 7:45 am)
Re: [GIT]: Networking, Patrick McHardy, (Mon Jul 21, 8:05 am)
Re: [GIT]: Networking, David Miller, (Mon Jul 21, 1:28 pm)
Re: [GIT]: Networking, Linus Torvalds, (Mon Jul 21, 1:40 pm)
Re: [GIT]: Networking, Patrick McHardy, (Mon Jul 21, 4:33 pm)
Re: [GIT]: Networking, David Miller, (Wed Jul 23, 7:42 pm)
Re: [GIT]: Networking, Stefan Richter, (Mon Jul 21, 7:28 am)
Re: [GIT]: Networking, Alexey Dobriyan, (Sun Jul 20, 9:09 pm)
Re: [GIT]: Networking, David Miller, (Sun Jul 20, 9:14 pm)
Re: [GIT]: Networking, Alexey Dobriyan, (Sun Jul 20, 10:40 pm)
Re: [GIT]: Networking, David Miller, (Sun Jul 20, 10:48 pm)
Re: [GIT]: Networking, David Miller, (Mon Jul 21, 1:11 am)
Re: [GIT]: Networking, Linus Torvalds, (Mon Jul 21, 12:49 pm)
Re: [GIT]: Networking, David Miller, (Mon Jul 21, 12:53 pm)
Re: [GIT]: Networking, Alexey Dobriyan, (Mon Jul 21, 7:57 am)
Re: [GIT]: Networking, David Miller, (Mon Jul 21, 11:27 am)
Re: [GIT]: Networking, Alexander Beregalov, (Mon Jul 21, 5:48 am)
Re: [GIT]: Networking, Ben Hutchings, (Mon Jul 21, 6:16 am)
Re: [GIT]: Networking, David Miller, (Mon Jul 21, 11:35 am)
Re: [GIT]: Networking, Alexander Beregalov, (Mon Jul 21, 12:04 pm)
Re: [GIT]: Networking, Alexey Dobriyan, (Sun Jul 20, 9:22 pm)