lkml.org 
[lkml]   [2002]   [Sep]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC][PATCH] sard changes for 2.5.34


On Thu, 12 Sep 2002, Rick Lindsley wrote:

> OK, that's a start. I think there was some work done on making
> kernel_stat percpu as well.
>
> Yes there's work on a couple of different fronts there. There is work
> to specifically make disk stats per cpu (actually, I have some 2.4
> patches already I could port), and there is a more general interface
> (statctr_t) which Dipankar Sarma (dipankar@in.ibm.com) is working on
> for 2.5 for stat counters in general which generalizes the per-cpu
> concept.

OK. First of all, I have serious problems with collecting all these
stats to hd_struct. Reasons:
a) _all_ stats you collect by request will be for entire disk.
By the time when request had been created, we simply don't remember
which partition it used to be. Pretending that they are per-partition
only obfuscates the things. Stats by bio are per-partition, but... see below
b) hd_struct had been completely removed from drivers (they neither
know nor care about it) and it will be completely gone RSN. There will be
a replacement, obviously, but it's not even certain that it will be a
single array.

> That would be great ... but I want to be sure we don't take so long
> working on the polish that we miss 10/31 with the main event. I can
> spend a few days incorporating all of these things and repost, if you
> don't think it makes it "too many changes at one time."

It might. There is a big rewrite of that area going on right now.
In particular, we are getting to the point when _all_ block devices are
going to acquire gendisks - partitioned or not. Next step after that
will be sane refcounting for gendisks and once that is done we are getting
rid of get_gendisk() for good - pointer will be cached in struct block_device
while it's open _and_ it will replace ->rq_dev in struct request.

These changes alone can cause a lot of fun, and then there is
reorganization of struct gendisk itself...

-
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-03-22 13:28    [W:0.409 / U:0.072 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site