* [PATCH] Enable /proc/diskstats for /dev/ramX
@ 2012-11-19 7:16 Ryo Onodera
2012-11-19 7:26 ` Ryo Onodera
0 siblings, 1 reply; 4+ messages in thread
From: Ryo Onodera @ 2012-11-19 7:16 UTC (permalink / raw)
To: Nick Piggin; +Cc: linux-kernel
Signed-off-by: Ryo Onodera <ryoqun@gmail.com>
---
drivers/block/brd.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/block/brd.c b/drivers/block/brd.c
index 531ceb3..181b2d7 100644
--- a/drivers/block/brd.c
+++ b/drivers/block/brd.c
@@ -332,6 +332,7 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
sector_t sector;
int i;
int err = -EIO;
+ int cpu;
sector = bio->bi_sector;
if (sector + (bio->bi_size >> SECTOR_SHIFT) >
@@ -348,6 +349,12 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
if (rw == READA)
rw = READ;
+ cpu = part_stat_lock();
+ part_stat_inc(cpu, &brd->brd_disk->part0, ios[rw]);
+ part_stat_add(cpu, &brd->brd_disk->part0, sectors[rw],
+ bio_sectors(bio));
+ part_stat_unlock();
+
bio_for_each_segment(bvec, bio, i) {
unsigned int len = bvec->bv_len;
err = brd_do_bvec(brd, bvec->bv_page, len,
--
1.7.9.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] Enable /proc/diskstats for /dev/ramX
2012-11-19 7:16 [PATCH] Enable /proc/diskstats for /dev/ramX Ryo Onodera
@ 2012-11-19 7:26 ` Ryo Onodera
2012-12-23 21:41 ` [PATCH] brd: enable diskstats Ryo Onodera
0 siblings, 1 reply; 4+ messages in thread
From: Ryo Onodera @ 2012-11-19 7:26 UTC (permalink / raw)
To: Nick Piggin; +Cc: linux-kernel
Hi,
I'm Ryo Onodera. I created this very tiny patch to enable /proc/diskstats for
/dev/ramX.
I'm contacting Nick Piggin as the maintainer of ramdisk ram block.
This is my first patch to the Linux kernel. Sorry if this is not the expected
way of submitting patches.
Regards
Ryo Onodera
On Mon, Nov 19, 2012 at 4:16 PM, Ryo Onodera <ryoqun@gmail.com> wrote:
> Signed-off-by: Ryo Onodera <ryoqun@gmail.com>
> ---
> drivers/block/brd.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/block/brd.c b/drivers/block/brd.c
> index 531ceb3..181b2d7 100644
> --- a/drivers/block/brd.c
> +++ b/drivers/block/brd.c
> @@ -332,6 +332,7 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
> sector_t sector;
> int i;
> int err = -EIO;
> + int cpu;
>
> sector = bio->bi_sector;
> if (sector + (bio->bi_size >> SECTOR_SHIFT) >
> @@ -348,6 +349,12 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
> if (rw == READA)
> rw = READ;
>
> + cpu = part_stat_lock();
> + part_stat_inc(cpu, &brd->brd_disk->part0, ios[rw]);
> + part_stat_add(cpu, &brd->brd_disk->part0, sectors[rw],
> + bio_sectors(bio));
> + part_stat_unlock();
> +
> bio_for_each_segment(bvec, bio, i) {
> unsigned int len = bvec->bv_len;
> err = brd_do_bvec(brd, bvec->bv_page, len,
> --
> 1.7.9.5
>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] brd: enable diskstats
2012-11-19 7:26 ` Ryo Onodera
@ 2012-12-23 21:41 ` Ryo Onodera
2012-12-23 21:47 ` Ryo Onodera
0 siblings, 1 reply; 4+ messages in thread
From: Ryo Onodera @ 2012-12-23 21:41 UTC (permalink / raw)
To: Nick Piggin; +Cc: linux-kernel, Namhyung Kim, Jens Axboe
Signed-off-by: Ryo Onodera <ryoqun@gmail.com>
---
drivers/block/brd.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/block/brd.c b/drivers/block/brd.c
index 531ceb3..181b2d7 100644
--- a/drivers/block/brd.c
+++ b/drivers/block/brd.c
@@ -332,6 +332,7 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
sector_t sector;
int i;
int err = -EIO;
+ int cpu;
sector = bio->bi_sector;
if (sector + (bio->bi_size >> SECTOR_SHIFT) >
@@ -348,6 +349,12 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
if (rw == READA)
rw = READ;
+ cpu = part_stat_lock();
+ part_stat_inc(cpu, &brd->brd_disk->part0, ios[rw]);
+ part_stat_add(cpu, &brd->brd_disk->part0, sectors[rw],
+ bio_sectors(bio));
+ part_stat_unlock();
+
bio_for_each_segment(bvec, bio, i) {
unsigned int len = bvec->bv_len;
err = brd_do_bvec(brd, bvec->bv_page, len,
--
1.7.9.5
On Mon, 2012-11-19 at 16:26 +0900, Ryo Onodera wrote:
> Hi,
>
> I'm Ryo Onodera. I created this very tiny patch to enable /proc/diskstats for
> /dev/ramX.
>
> I'm contacting Nick Piggin as the maintainer of ramdisk ram block.
>
> This is my first patch to the Linux kernel. Sorry if this is not the expected
> way of submitting patches.
>
> Regards
>
> Ryo Onodera
>
> On Mon, Nov 19, 2012 at 4:16 PM, Ryo Onodera <ryoqun@gmail.com> wrote:
> > Signed-off-by: Ryo Onodera <ryoqun@gmail.com>
> > ---
> > drivers/block/brd.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/drivers/block/brd.c b/drivers/block/brd.c
> > index 531ceb3..181b2d7 100644
> > --- a/drivers/block/brd.c
> > +++ b/drivers/block/brd.c
> > @@ -332,6 +332,7 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
> > sector_t sector;
> > int i;
> > int err = -EIO;
> > + int cpu;
> >
> > sector = bio->bi_sector;
> > if (sector + (bio->bi_size >> SECTOR_SHIFT) >
> > @@ -348,6 +349,12 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
> > if (rw == READA)
> > rw = READ;
> >
> > + cpu = part_stat_lock();
> > + part_stat_inc(cpu, &brd->brd_disk->part0, ios[rw]);
> > + part_stat_add(cpu, &brd->brd_disk->part0, sectors[rw],
> > + bio_sectors(bio));
> > + part_stat_unlock();
> > +
> > bio_for_each_segment(bvec, bio, i) {
> > unsigned int len = bvec->bv_len;
> > err = brd_do_bvec(brd, bvec->bv_page, len,
> > --
> > 1.7.9.5
> >
> >
> >
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] brd: enable diskstats
2012-12-23 21:41 ` [PATCH] brd: enable diskstats Ryo Onodera
@ 2012-12-23 21:47 ` Ryo Onodera
0 siblings, 0 replies; 4+ messages in thread
From: Ryo Onodera @ 2012-12-23 21:47 UTC (permalink / raw)
To: Nick Piggin; +Cc: linux-kernel, Namhyung Kim, Jens Axboe
Hi,
I'm Ryo Onodera.
This is a resend for my earlier patch. Because I got no reply, I've
added other people as CC, picking from the commit history of the
relevant file.
I created this very tiny patch to enable /proc/diskstats for /dev/ramX.
I'm contacting Nick Piggin as the maintainer of ramdisk ram block. As
mentioned above, I'm also contacting some related people to get more
attention.
This is my first patch to the Linux kernel. Sorry if this is not the
expected way of submitting patches.
Regards
On Mon, 2012-12-24 at 06:41 +0900, Ryo Onodera wrote:
> Signed-off-by: Ryo Onodera <ryoqun@gmail.com>
> ---
> drivers/block/brd.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/block/brd.c b/drivers/block/brd.c
> index 531ceb3..181b2d7 100644
> --- a/drivers/block/brd.c
> +++ b/drivers/block/brd.c
> @@ -332,6 +332,7 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
> sector_t sector;
> int i;
> int err = -EIO;
> + int cpu;
>
> sector = bio->bi_sector;
> if (sector + (bio->bi_size >> SECTOR_SHIFT) >
> @@ -348,6 +349,12 @@ static void brd_make_request(struct request_queue *q, struct bio *bio)
> if (rw == READA)
> rw = READ;
>
> + cpu = part_stat_lock();
> + part_stat_inc(cpu, &brd->brd_disk->part0, ios[rw]);
> + part_stat_add(cpu, &brd->brd_disk->part0, sectors[rw],
> + bio_sectors(bio));
> + part_stat_unlock();
> +
> bio_for_each_segment(bvec, bio, i) {
> unsigned int len = bvec->bv_len;
> err = brd_do_bvec(brd, bvec->bv_page, len,
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-12-23 21:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-19 7:16 [PATCH] Enable /proc/diskstats for /dev/ramX Ryo Onodera
2012-11-19 7:26 ` Ryo Onodera
2012-12-23 21:41 ` [PATCH] brd: enable diskstats Ryo Onodera
2012-12-23 21:47 ` Ryo Onodera
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.