linux-kernel - Re: [PATCH 2/2] drivers: xen/block: add multi-page ring support

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150206110134.GC30821@zion.uk.xensource.com>
Date:	Fri, 6 Feb 2015 11:01:34 +0000
From:	Wei Liu <wei.liu2@...rix.com>
To:	Bob Liu <bob.liu@...cle.com>
CC:	Roger Pau Monné <roger.pau@...rix.com>,
	<xen-devel@...ts.xen.org>, <david.vrabel@...rix.com>,
	<linux-kernel@...r.kernel.org>, <konrad.wilk@...cle.com>,
	<boris.ostrovsky@...cle.com>, <wei.liu2@...rix.com>
Subject: Re: [PATCH 2/2] drivers: xen/block: add multi-page ring support

On Fri, Feb 06, 2015 at 06:47:17PM +0800, Bob Liu wrote:
> 
> On 02/02/2015 06:43 PM, Roger Pau Monné wrote:
> > El 23/01/15 a les 10.14, Bob Liu ha escrit:
> >> Extend xen/block to support multi-page ring.
> >>  * xen-blkback notify blkfront with feature-multi-ring-pages
> >>  * xen-blkfront write to xenstore about how many pages are used as the ring
> >>
> >> If using 4 pages as the ring, inflight requests inscreased from 32 to 128 and
> >> IOPS improved nearly 400% on our system.
> >>
> >> Signed-off-by: Bob Liu <bob.liu@...cle.com>
> >> ---
> >>  drivers/block/xen-blkback/xenbus.c |   86 +++++++++++++++++++++++++--------
> >>  drivers/block/xen-blkfront.c       |   94 ++++++++++++++++++++++++++----------
> > 
> > This requires a patch to xen/include/public/io/blkif.h in the Xen
> > repository describing how this protocol extension is going to work
> > before reviewing the Linux implementation.
> > 
> 
> But there is no protocol change, just add an parameter "max_ring_pages"
> to control how many pages can be used as the ring. Now with one page we
> can only have 32 requests.
> 

I think Roger meant the xenbus protocol used to negotiate this feature,
not the data protocol regarding how request and response are
interpreted. Blkif.h (and netif.h FWIW) has section to describe xenbus
protocol.

Wei.

> -- 
> Regards,
> -Bob
--
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/