[v2] Vendor stats support in vdpasim_net

[PATCH V2 0/4] Vendor stats support in vdpasim_net

Jason Wang posted 4 patches 2 years, 8 months ago

Diff against v1 v3
Download series mbox

There is a newer version of this series

drivers/vdpa/vdpa_sim/vdpa_sim.c     |  33 +++-
drivers/vdpa/vdpa_sim/vdpa_sim.h     |   4 +
drivers/vdpa/vdpa_sim/vdpa_sim_blk.c |   1 +
drivers/vdpa/vdpa_sim/vdpa_sim_net.c | 221 ++++++++++++++++++++++++++-
4 files changed, 248 insertions(+), 11 deletions(-)

Expand all Fold all

[PATCH V2 0/4] Vendor stats support in vdpasim_net

Posted by Jason Wang 2 years, 8 months ago

Hi All:

This series implements vendor stats in vdpasim_net. Please review.

Thanks

Changes since V1:
- typo fixes
- move the duplicated get_vendor_vq_stats() in
  vdpasim_batch_config_ops to vdpa_sim_config_ops
- use -EOPNOTSUPP instead of -EINVAL in vdpasim_get_vq_stats
- introdce a dedicated variable to record the successful cvq request
  and track the number of requests correctly

Jason Wang (4):
  vdpa_sim: switch to use __vdpa_alloc_device()
  vdpasim: customize allocation size
  vdpa_sim: support vendor statistics
  vdpa_sim_net: vendor satistics

 drivers/vdpa/vdpa_sim/vdpa_sim.c     |  33 +++-
 drivers/vdpa/vdpa_sim/vdpa_sim.h     |   4 +
 drivers/vdpa/vdpa_sim/vdpa_sim_blk.c |   1 +
 drivers/vdpa/vdpa_sim/vdpa_sim_net.c | 221 ++++++++++++++++++++++++++-
 4 files changed, 248 insertions(+), 11 deletions(-)

-- 
2.25.1

Re: [PATCH V2 0/4] Vendor stats support in vdpasim_net

Posted by Eugenio Perez Martin 2 years, 8 months ago

On Thu, Dec 22, 2022 at 6:01 AM Jason Wang <jasowang@redhat.com> wrote:
>
> Hi All:
>
> This series implements vendor stats in vdpasim_net. Please review.
>
> Thanks
>

Apart from the commented blanks,

Acked-by: Eugenio Pérez <eperezma@redhat.com>

Thanks!

> Changes since V1:
> - typo fixes
> - move the duplicated get_vendor_vq_stats() in
>   vdpasim_batch_config_ops to vdpa_sim_config_ops
> - use -EOPNOTSUPP instead of -EINVAL in vdpasim_get_vq_stats
> - introdce a dedicated variable to record the successful cvq request
>   and track the number of requests correctly
>
> Jason Wang (4):
>   vdpa_sim: switch to use __vdpa_alloc_device()
>   vdpasim: customize allocation size
>   vdpa_sim: support vendor statistics
>   vdpa_sim_net: vendor satistics
>
>  drivers/vdpa/vdpa_sim/vdpa_sim.c     |  33 +++-
>  drivers/vdpa/vdpa_sim/vdpa_sim.h     |   4 +
>  drivers/vdpa/vdpa_sim/vdpa_sim_blk.c |   1 +
>  drivers/vdpa/vdpa_sim/vdpa_sim_net.c | 221 ++++++++++++++++++++++++++-
>  4 files changed, 248 insertions(+), 11 deletions(-)
>
> --
> 2.25.1
>