Re: [PATCH RFC v2 08/10] vdpa: add vdpa simulator for block device

[Stefano Garzarella]

On Sun, Jan 31, 2021 at 05:31:43PM +0200, Max Gurtovoy wrote:
> On 1/28/2021 4:41 PM, Stefano Garzarella wrote:
> > From: Max Gurtovoy <mgurtovoy@xxxxxxxxxx>
> >
> > This will allow running vDPA for virtio block protocol.
> >
> > Signed-off-by: Max Gurtovoy <mgurtovoy@xxxxxxxxxx>
> > [sgarzare: various cleanups/fixes]
> > Signed-off-by: Stefano Garzarella <sgarzare@xxxxxxxxxx>
> > ---
> > v2:
> > - rebased on top of other changes (dev_attr, get_config(), notify(), etc.)
> > - memset to 0 the config structure in vdpasim_blk_get_config()
> > - used vdpasim pointer in vdpasim_blk_get_config()
> >
> > v1:
> > - Removed unused headers
> > - Used cpu_to_vdpasim*() to store config fields
> > - Replaced 'select VDPA_SIM' with 'depends on VDPA_SIM' since selected
> >   option can not depend on other [Jason]
> > - Start with a single queue for now [Jason]
> > - Add comments to memory barriers
> > ---
> >  drivers/vdpa/vdpa_sim/vdpa_sim_blk.c | 145 +++++++++++++++++++++++++++
> >  drivers/vdpa/Kconfig                 |   7 ++
> >  drivers/vdpa/vdpa_sim/Makefile       |   1 +
> >  3 files changed, 153 insertions(+)
> >  create mode 100644 drivers/vdpa/vdpa_sim/vdpa_sim_blk.c
> >
> > diff --git a/drivers/vdpa/vdpa_sim/vdpa_sim_blk.c b/drivers/vdpa/vdpa_sim/vdpa_sim_blk.c
> > new file mode 100644
> > index 000000000000..999f9ca0b628
> > --- /dev/null
> > +++ b/drivers/vdpa/vdpa_sim/vdpa_sim_blk.c
> > @@ -0,0 +1,145 @@
> > +// SPDX-License-Identifier: GPL-2.0-only
> > +/*
> > + * VDPA simulator for block device.
> > + *
> > + * Copyright (c) 2020, Mellanox Technologies. All rights reserved.
>
> I guess we can change the copyright from Mellanox to:
>
> Copyright (c) 2020, NVIDIA CORPORATION. All rights reserved.

I'll update in the next version.

Thanks,
Stefano