Messages in this thread | | | Date | Sun, 29 Jan 2012 13:07:22 +0800 | From | Wu Fengguang <> | Subject | Re: [PATCH 6/9] readahead: add /debug/readahead/stats |
| |
On Fri, Jan 27, 2012 at 12:15:51PM -0800, Andrew Morton wrote:
> > The accounting code will be compiled in by default > > (CONFIG_READAHEAD_STATS=y), and will remain inactive by default. > > I agree with those choices. They effectively mean that the stats will > be a developer-only/debugger-only thing. So even if the atomic_inc() > costs are measurable during these develop/debug sessions, is anyone > likely to care?
Sorry I have changed the default to CONFIG_READAHEAD_STATS=n to avoid bloating the kernel (and forgot to edit the changelog accordingly).
I'm not sure how many people are going to check the readahead stats.
Thanks, Fengguang
| |