Re: [Xen-devel] [PATCH 2/3] blkif: enable multi-page ring supporting

2015-04-08 Thread Bob Liu
On 04/08/2015 10:25 PM, David Vrabel wrote: On 08/04/15 15:21, Konrad Rzeszutek Wilk wrote: On Wed, Apr 08, 2015 at 11:46:44AM +0100, David Vrabel wrote: On 03/04/15 07:45, Bob Liu wrote: Remove the 'DEPRECATED' and next patch will follow the protocol defined here to support multi-page

Re: [Xen-devel] [PATCH 2/3] blkif: enable multi-page ring supporting

2015-04-08 Thread David Vrabel
On 03/04/15 07:45, Bob Liu wrote: Remove the 'DEPRECATED' and next patch will follow the protocol defined here to support multi-page ring. Why are you un-deprecating this instead of using max-ring-page-order? Are you allowing for non power of two counts? If so, the documentation needs to be

Re: [Xen-devel] [PATCH 2/3] blkif: enable multi-page ring supporting

2015-04-08 Thread Konrad Rzeszutek Wilk
On Wed, Apr 08, 2015 at 11:46:44AM +0100, David Vrabel wrote: On 03/04/15 07:45, Bob Liu wrote: Remove the 'DEPRECATED' and next patch will follow the protocol defined here to support multi-page ring. Why are you un-deprecating this instead of using max-ring-page-order? Are you

Re: [Xen-devel] [PATCH 2/3] blkif: enable multi-page ring supporting

2015-04-08 Thread David Vrabel
On 08/04/15 15:21, Konrad Rzeszutek Wilk wrote: On Wed, Apr 08, 2015 at 11:46:44AM +0100, David Vrabel wrote: On 03/04/15 07:45, Bob Liu wrote: Remove the 'DEPRECATED' and next patch will follow the protocol defined here to support multi-page ring. Why are you un-deprecating this instead

[Xen-devel] [PATCH 2/3] blkif: enable multi-page ring supporting

2015-04-03 Thread Bob Liu
Remove the 'DEPRECATED' and next patch will follow the protocol defined here to support multi-page ring. Signed-off-by: Bob Liu bob@oracle.com --- xen/include/public/io/blkif.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/include/public/io/blkif.h