* [Qemu-devel] [PATCH 4/4] net, virtio_net: replace the magic value @ 2013-11-18 8:46 Zhi Yong Wu 2013-11-18 9:50 ` Michael S. Tsirkin 0 siblings, 1 reply; 5+ messages in thread From: Zhi Yong Wu @ 2013-11-18 8:46 UTC (permalink / raw) To: netdev; +Cc: qemu-devel, rusty, Zhi Yong Wu, linux-kernel, mst From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> It is more appropriate to use # of queue pairs currently used by the driver instead of a magic value. Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> --- drivers/net/virtio_net.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index cdc7c90..e0cb2d1 100644 --- a/drivers/net/virtio_net.c +++ b/drivers/net/virtio_net.c @@ -1619,8 +1619,8 @@ static int virtnet_probe(struct virtio_device *vdev) if (err) goto free_stats; - netif_set_real_num_tx_queues(dev, 1); - netif_set_real_num_rx_queues(dev, 1); + netif_set_real_num_tx_queues(dev, vi->curr_queue_pairs); + netif_set_real_num_rx_queues(dev, vi->curr_queue_pairs); err = register_netdev(dev); if (err) { -- 1.7.6.5 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH 4/4] net, virtio_net: replace the magic value 2013-11-18 8:46 [Qemu-devel] [PATCH 4/4] net, virtio_net: replace the magic value Zhi Yong Wu @ 2013-11-18 9:50 ` Michael S. Tsirkin 2013-11-18 10:07 ` Zhi Yong Wu 0 siblings, 1 reply; 5+ messages in thread From: Michael S. Tsirkin @ 2013-11-18 9:50 UTC (permalink / raw) To: Zhi Yong Wu; +Cc: netdev, rusty, Zhi Yong Wu, qemu-devel, linux-kernel On Mon, Nov 18, 2013 at 04:46:20PM +0800, Zhi Yong Wu wrote: > From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> > > It is more appropriate to use # of queue pairs currently used by > the driver instead of a magic value. > > Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> I don't mind, but driver should be submitted separately from qemu patches. As it is only patch 4/4 made it to netdev. > --- > drivers/net/virtio_net.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c > index cdc7c90..e0cb2d1 100644 > --- a/drivers/net/virtio_net.c > +++ b/drivers/net/virtio_net.c > @@ -1619,8 +1619,8 @@ static int virtnet_probe(struct virtio_device *vdev) > if (err) > goto free_stats; > > - netif_set_real_num_tx_queues(dev, 1); > - netif_set_real_num_rx_queues(dev, 1); > + netif_set_real_num_tx_queues(dev, vi->curr_queue_pairs); > + netif_set_real_num_rx_queues(dev, vi->curr_queue_pairs); > > err = register_netdev(dev); > if (err) { > -- > 1.7.6.5 > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH 4/4] net, virtio_net: replace the magic value 2013-11-18 9:50 ` Michael S. Tsirkin @ 2013-11-18 10:07 ` Zhi Yong Wu 2013-11-18 10:15 ` Michael S. Tsirkin 0 siblings, 1 reply; 5+ messages in thread From: Zhi Yong Wu @ 2013-11-18 10:07 UTC (permalink / raw) To: Michael S. Tsirkin Cc: netdev, Rusty Russell, Zhi Yong Wu, QEMU Developers, linux-kernel mlist On Mon, Nov 18, 2013 at 5:50 PM, Michael S. Tsirkin <mst@redhat.com> wrote: > On Mon, Nov 18, 2013 at 04:46:20PM +0800, Zhi Yong Wu wrote: >> From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> >> >> It is more appropriate to use # of queue pairs currently used by >> the driver instead of a magic value. >> >> Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> > > I don't mind, but driver should be submitted separately > from qemu patches. As it is only patch 4/4 made it to netdev. ok, i will sent v2. By the way, can you help take a look at the following patches? Maybe i can send their v2 together. [PATCH 1/3] vhost: remove the dead branch [PATCH 2/3] vhost: adjust vhost_dev_init() to be void [PATCH 3/3] vhost: fix the wrong log descriptions > >> --- >> drivers/net/virtio_net.c | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c >> index cdc7c90..e0cb2d1 100644 >> --- a/drivers/net/virtio_net.c >> +++ b/drivers/net/virtio_net.c >> @@ -1619,8 +1619,8 @@ static int virtnet_probe(struct virtio_device *vdev) >> if (err) >> goto free_stats; >> >> - netif_set_real_num_tx_queues(dev, 1); >> - netif_set_real_num_rx_queues(dev, 1); >> + netif_set_real_num_tx_queues(dev, vi->curr_queue_pairs); >> + netif_set_real_num_rx_queues(dev, vi->curr_queue_pairs); >> >> err = register_netdev(dev); >> if (err) { >> -- >> 1.7.6.5 >> -- Regards, Zhi Yong Wu ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH 4/4] net, virtio_net: replace the magic value 2013-11-18 10:07 ` Zhi Yong Wu @ 2013-11-18 10:15 ` Michael S. Tsirkin 2013-11-18 12:26 ` Zhi Yong Wu 0 siblings, 1 reply; 5+ messages in thread From: Michael S. Tsirkin @ 2013-11-18 10:15 UTC (permalink / raw) To: Zhi Yong Wu Cc: netdev, Rusty Russell, Zhi Yong Wu, QEMU Developers, linux-kernel mlist On Mon, Nov 18, 2013 at 06:07:45PM +0800, Zhi Yong Wu wrote: > On Mon, Nov 18, 2013 at 5:50 PM, Michael S. Tsirkin <mst@redhat.com> wrote: > > On Mon, Nov 18, 2013 at 04:46:20PM +0800, Zhi Yong Wu wrote: > >> From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> > >> > >> It is more appropriate to use # of queue pairs currently used by > >> the driver instead of a magic value. > >> > >> Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> > > > > I don't mind, but driver should be submitted separately > > from qemu patches. As it is only patch 4/4 made it to netdev. > ok, i will sent v2. By the way, can you help take a look at the > following patches? Will do. > Maybe i can send their v2 together. Please don't, these seem to be completely unrelated. > [PATCH 1/3] vhost: remove the dead branch > [PATCH 2/3] vhost: adjust vhost_dev_init() to be void > [PATCH 3/3] vhost: fix the wrong log descriptions > > > > > >> --- > >> drivers/net/virtio_net.c | 4 ++-- > >> 1 files changed, 2 insertions(+), 2 deletions(-) > >> > >> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c > >> index cdc7c90..e0cb2d1 100644 > >> --- a/drivers/net/virtio_net.c > >> +++ b/drivers/net/virtio_net.c > >> @@ -1619,8 +1619,8 @@ static int virtnet_probe(struct virtio_device *vdev) > >> if (err) > >> goto free_stats; > >> > >> - netif_set_real_num_tx_queues(dev, 1); > >> - netif_set_real_num_rx_queues(dev, 1); > >> + netif_set_real_num_tx_queues(dev, vi->curr_queue_pairs); > >> + netif_set_real_num_rx_queues(dev, vi->curr_queue_pairs); > >> > >> err = register_netdev(dev); > >> if (err) { > >> -- > >> 1.7.6.5 > >> > > > > -- > Regards, > > Zhi Yong Wu ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH 4/4] net, virtio_net: replace the magic value 2013-11-18 10:15 ` Michael S. Tsirkin @ 2013-11-18 12:26 ` Zhi Yong Wu 0 siblings, 0 replies; 5+ messages in thread From: Zhi Yong Wu @ 2013-11-18 12:26 UTC (permalink / raw) To: Michael S. Tsirkin Cc: netdev, Rusty Russell, Zhi Yong Wu, QEMU Developers, linux-kernel mlist On Mon, Nov 18, 2013 at 6:15 PM, Michael S. Tsirkin <mst@redhat.com> wrote: > On Mon, Nov 18, 2013 at 06:07:45PM +0800, Zhi Yong Wu wrote: >> On Mon, Nov 18, 2013 at 5:50 PM, Michael S. Tsirkin <mst@redhat.com> wrote: >> > On Mon, Nov 18, 2013 at 04:46:20PM +0800, Zhi Yong Wu wrote: >> >> From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> >> >> >> >> It is more appropriate to use # of queue pairs currently used by >> >> the driver instead of a magic value. >> >> >> >> Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com> >> > >> > I don't mind, but driver should be submitted separately >> > from qemu patches. As it is only patch 4/4 made it to netdev. >> ok, i will sent v2. By the way, can you help take a look at the >> following patches? > > Will do. > >> Maybe i can send their v2 together. > > Please don't, these seem to be completely unrelated. OK, i will send it separately. > >> [PATCH 1/3] vhost: remove the dead branch >> [PATCH 2/3] vhost: adjust vhost_dev_init() to be void >> [PATCH 3/3] vhost: fix the wrong log descriptions >> >> >> > >> >> --- >> >> drivers/net/virtio_net.c | 4 ++-- >> >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> >> >> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c >> >> index cdc7c90..e0cb2d1 100644 >> >> --- a/drivers/net/virtio_net.c >> >> +++ b/drivers/net/virtio_net.c >> >> @@ -1619,8 +1619,8 @@ static int virtnet_probe(struct virtio_device *vdev) >> >> if (err) >> >> goto free_stats; >> >> >> >> - netif_set_real_num_tx_queues(dev, 1); >> >> - netif_set_real_num_rx_queues(dev, 1); >> >> + netif_set_real_num_tx_queues(dev, vi->curr_queue_pairs); >> >> + netif_set_real_num_rx_queues(dev, vi->curr_queue_pairs); >> >> >> >> err = register_netdev(dev); >> >> if (err) { >> >> -- >> >> 1.7.6.5 >> >> >> >> >> >> -- >> Regards, >> >> Zhi Yong Wu -- Regards, Zhi Yong Wu ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-11-18 12:26 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-11-18 8:46 [Qemu-devel] [PATCH 4/4] net, virtio_net: replace the magic value Zhi Yong Wu 2013-11-18 9:50 ` Michael S. Tsirkin 2013-11-18 10:07 ` Zhi Yong Wu 2013-11-18 10:15 ` Michael S. Tsirkin 2013-11-18 12:26 ` Zhi Yong Wu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).