[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20071222123705.GA14767@one.firstfloor.org>
Date: Sat, 22 Dec 2007 13:37:05 +0100
From: Andi Kleen <andi@...stfloor.org>
To: Ingo Molnar <mingo@...e.hu>
Cc: Andi Kleen <andi@...stfloor.org>,
Christoph Lameter <clameter@....com>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Steven Rostedt <rostedt@...dmis.org>,
LKML <linux-kernel@...r.kernel.org>,
Andrew Morton <akpm@...ux-foundation.org>,
Christoph Hellwig <hch@...radead.org>,
"Rafael J. Wysocki" <rjw@...k.pl>
Subject: Re: Major regression on hackbench with SLUB (more numbers)
> could be offered on SLUB too.
Sure (I argued that in a earlier mail in fact), but it doesn't make
sense to fake into the old slabinfo format.
>
> 'top' isnt critical functionality either like udev, and the ABI does not
> only cover 'critical' functionality. A utility suddenly not working
> gives Linux a pretty amateurish feeling. Should we tell users/admins:
> "Hehe, gotcha! Didnt you know /proc/slabinfo was not an ABI? Poor sob.
> If you want your stuff to continue working, use Windows next time around
> or what. Sheesh, what do these people want!' ;-)
While I sympatise with this point in practice non critial ABIs change
regularly (and sometimes even critical ones like sched_yield ...)
>
> the rule is very simple: unless you have really, really, really, REALLY
> good reasons, just dont break stuff.
Removing an interface that exposes lots of internal details when you
rewrite the subsystem and those internal details all change seems
like a good reason to me.
Besides the original interface was broken anyways. The version
number was one of the interface worst ideas ever imho and slabtop's handling
of it quite dumb. The better way would have been to always add new
fields and never remove old ones. Hopefully the new /sys based interface
will be better.
-Andi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists