Messages in this thread | | | Date | Fri, 22 Jun 2007 21:21:25 +0100 (BST) | From | Hugh Dickins <> | Subject | Re: Oops in a driver while using SLUB as a SLAB allocator |
| |
On Fri, 22 Jun 2007, Christoph Lameter wrote: > > We need to fix any remaining weird slab object uses right now. Your check > leaves a lot of holes open. 2.6.22 removes all other such strange slab > uses in other arches. It would be inconsistent if we left these things in > ARM (and maybe PA-RISC).
As I understand it, that driver used to work right with SLAB, then oopsed with SLUB, and now works okay again with the page_mapping patch?
I'm unclear how it comes about that you removed "all other such strange slab uses in other arches", yet missed this? That suggests there may be further unexpected uses.
It worries me that any use which catches you by surprise has to be fixed up in the caller, rather than in slub itself: slab/slub is a service, not a master. But I'm rather repeating myself.
Hugh - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |