linux-kernel - Re: [PATCH] brd: enable diskstats

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <1356299260.9057.8.camel@erika>
Date:	Mon, 24 Dec 2012 06:47:40 +0900
From:	Ryo Onodera <ryoqun@...il.com>
To:	Nick Piggin <npiggin@...nel.dk>
Cc:	linux-kernel@...r.kernel.org, Namhyung Kim <namhyung@...il.com>,
	Jens Axboe <jaxboe@...ionio.com>
Subject: Re: [PATCH] brd: enable diskstats

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@...il.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,


--
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/