Re: [PATCH v4 4/5] block: drop bdrv_prwv

2020-05-27 Thread Vladimir Sementsov-Ogievskiy
27.05.2020 00:34, Eric Blake wrote: On 5/25/20 5:08 AM, Vladimir Sementsov-Ogievskiy wrote: Now, when we are not more paying extra code for coroutine wrappers, there no more sence in extra indirection layer: bdrv_prwv(). Let's drop it and instead genereate pure bdrv_preadv() and bdrv_pwritev().

Re: [PATCH v4 4/5] block: drop bdrv_prwv

2020-05-26 Thread Eric Blake
On 5/25/20 5:08 AM, Vladimir Sementsov-Ogievskiy wrote: Now, when we are not more paying extra code for coroutine wrappers, there no more sence in extra indirection layer: bdrv_prwv(). Let's drop it and instead genereate pure bdrv_preadv() and bdrv_pwritev(). Typos and grammar; I suggest: Now

[PATCH v4 4/5] block: drop bdrv_prwv

2020-05-25 Thread Vladimir Sementsov-Ogievskiy
Now, when we are not more paying extra code for coroutine wrappers, there no more sence in extra indirection layer: bdrv_prwv(). Let's drop it and instead genereate pure bdrv_preadv() and bdrv_pwritev(). Currently, bdrv_pwritev() and bdrv_preadv() are returning bytes on success, auto generated fun