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

From: Eugenio Perez Martin
Date: Thu Dec 22 2022 - 03:51:39 EST


On Thu, Dec 22, 2022 at 6:01 AM Jason Wang <jasowang@xxxxxxxxxx> 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@xxxxxxxxxx>

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
>