[0/4] virtio-blk: small cleanup

Message

Chaitanya Kulkarni April 20, 2022, 4:10 a.m. UTC

Hi,

This has some nit fixes and code cleanups along with removing
deprecated API fir ida_simple_XXX().

-ck

Chaitanya Kulkarni (4):
  virtio-blk: remove additional check in fast path
  virtio-blk: don't add a new line
  virtio-blk: avoid function call in the fast path
  virtio-blk: remove deprecated ida_simple_XXX()

 drivers/block/virtio_blk.c | 38 +++++++++++++++++---------------------
 1 file changed, 17 insertions(+), 21 deletions(-)

Comments

Chaitanya Kulkarni April 21, 2022, 9:56 p.m. UTC | #1

On 4/19/22 21:10, Chaitanya Kulkarni wrote:
> Hi,
> 
> This has some nit fixes and code cleanups along with removing
> deprecated API fir ida_simple_XXX().
> 
> -ck
> 
> Chaitanya Kulkarni (4):
>    virtio-blk: remove additional check in fast path
>    virtio-blk: don't add a new line
>    virtio-blk: avoid function call in the fast path
>    virtio-blk: remove deprecated ida_simple_XXX()
> 
>   drivers/block/virtio_blk.c | 38 +++++++++++++++++---------------------
>   1 file changed, 17 insertions(+), 21 deletions(-)
> 

Thanks for the review, I'll send out a V2 and drop patches that
lacks the quantitative data..

-ck

Stefan Hajnoczi April 22, 2022, 2:56 p.m. UTC | #2

On Thu, Apr 21, 2022 at 09:56:28PM +0000, Chaitanya Kulkarni wrote:
> On 4/19/22 21:10, Chaitanya Kulkarni wrote:
> > Hi,
> > 
> > This has some nit fixes and code cleanups along with removing
> > deprecated API fir ida_simple_XXX().
> > 
> > -ck
> > 
> > Chaitanya Kulkarni (4):
> >    virtio-blk: remove additional check in fast path
> >    virtio-blk: don't add a new line
> >    virtio-blk: avoid function call in the fast path
> >    virtio-blk: remove deprecated ida_simple_XXX()
> > 
> >   drivers/block/virtio_blk.c | 38 +++++++++++++++++---------------------
> >   1 file changed, 17 insertions(+), 21 deletions(-)
> > 
> 
> Thanks for the review, I'll send out a V2 and drop patches that
> lacks the quantitative data..

Thank you!

Stefan