[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20210314045559-mutt-send-email-mst@kernel.org>
Date: Sun, 14 Mar 2021 04:56:04 -0400
From: "Michael S. Tsirkin" <mst@...hat.com>
To: Alexander Duyck <alexander.duyck@...il.com>
Cc: davem@...emloft.net, kuba@...nel.org, netdev@...r.kernel.org,
oss-drivers@...ronome.com, simon.horman@...ronome.com,
yisen.zhuang@...wei.com, salil.mehta@...wei.com,
intel-wired-lan@...ts.osuosl.org, jesse.brandeburg@...el.com,
anthony.l.nguyen@...el.com, drivers@...sando.io,
snelson@...sando.io, netanel@...zon.com, akiyano@...zon.com,
gtzalik@...zon.com, saeedb@...zon.com,
GR-Linux-NIC-Dev@...vell.com, skalluru@...vell.com,
rmody@...vell.com, kys@...rosoft.com, haiyangz@...rosoft.com,
sthemmin@...rosoft.com, wei.liu@...nel.org, jasowang@...hat.com,
pv-drivers@...are.com, doshir@...are.com, alexanderduyck@...com,
Kernel-team@...com
Subject: Re: [net-next PATCH 07/10] virtio_net: Update driver to use
ethtool_sprintf
On Fri, Mar 12, 2021 at 09:48:46AM -0800, Alexander Duyck wrote:
> From: Alexander Duyck <alexanderduyck@...com>
>
> Update the code to replace instances of snprintf and a pointer update with
> just calling ethtool_sprintf.
>
> Also replace the char pointer with a u8 pointer to avoid having to recast
> the pointer type.
>
> Signed-off-by: Alexander Duyck <alexanderduyck@...com>
Acked-by: Michael S. Tsirkin <mst@...hat.com>
> ---
> drivers/net/virtio_net.c | 18 +++++++-----------
> 1 file changed, 7 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
> index e97288dd6e5a..77ba8e2fc11c 100644
> --- a/drivers/net/virtio_net.c
> +++ b/drivers/net/virtio_net.c
> @@ -2138,25 +2138,21 @@ static int virtnet_set_channels(struct net_device *dev,
> static void virtnet_get_strings(struct net_device *dev, u32 stringset, u8 *data)
> {
> struct virtnet_info *vi = netdev_priv(dev);
> - char *p = (char *)data;
> unsigned int i, j;
> + u8 *p = data;
>
> switch (stringset) {
> case ETH_SS_STATS:
> for (i = 0; i < vi->curr_queue_pairs; i++) {
> - for (j = 0; j < VIRTNET_RQ_STATS_LEN; j++) {
> - snprintf(p, ETH_GSTRING_LEN, "rx_queue_%u_%s",
> - i, virtnet_rq_stats_desc[j].desc);
> - p += ETH_GSTRING_LEN;
> - }
> + for (j = 0; j < VIRTNET_RQ_STATS_LEN; j++)
> + ethtool_sprintf(&p, "rx_queue_%u_%s", i,
> + virtnet_rq_stats_desc[j].desc);
> }
>
> for (i = 0; i < vi->curr_queue_pairs; i++) {
> - for (j = 0; j < VIRTNET_SQ_STATS_LEN; j++) {
> - snprintf(p, ETH_GSTRING_LEN, "tx_queue_%u_%s",
> - i, virtnet_sq_stats_desc[j].desc);
> - p += ETH_GSTRING_LEN;
> - }
> + for (j = 0; j < VIRTNET_SQ_STATS_LEN; j++)
> + ethtool_sprintf(&p, "tx_queue_%u_%s", i,
> + virtnet_sq_stats_desc[j].desc);
> }
> break;
> }
>
Powered by blists - more mailing lists