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

[Bob Liu <bob.liu@oracle.com>]

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 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 updated as well.
>>>
>>> Also num-ring-pages is also tagged as deprecated.
>>>
>>> Changes in blkif.h need to be made in Xen first.
>>
>> Um, this patch is against Xen tree.
> 
> With this being patch #2 in a Linux series no one else is going to
> notice this is Xen patch either.
> 

Sorry, I'll separate this one.

Could you please also review the [PATCH v3 3/3] xen/block: add
multi-page ring support?

Thanks,
-Bob