lkml.org 
[lkml]   [2005]   [Dec]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 0/4] linux-2.6-block: deactivating pagecache for benchmarks
Date
Dirk Henning Gerdes <mail@dirk-gerdes.de> wrote:

> For doing benchmarks on the I/O-Schedulers, I thought it would be very
> useful to disable the pagecache.
>
> I didn't want to make it so complicated so I just mark pages as
> not-uptodate, so they have to be read again. Another reason was, that I
> wanted to keep the conditions as near to reality as possible.
>
> Further I thought it would be useful, if you could turn the pagecache on
> and off without rebooting the system.
>
> I implemented a proc-fs entry "/proc/benchmark/pagecache" for this.

1) This mail is the only documentation on how to operate your patch.
How do you suppose your users to find out how to operate the switch?
(I asume it's really a switch, a toggle would be insane.)

Since it's very short and only for special purpose, documenting it
in Kconfig mignt be enough.

2) You're seperating your patches by file, not by function. ungood.

3) Your patches introduce a lot of whitespace.
--
Ich danke GMX dafür, die Verwendung meiner Adressen mittels per SPF
verbreiteten Lügen zu sabotieren.
-
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/

\
 
 \ /
  Last update: 2005-12-01 23:49    [W:0.275 / U:0.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site