All of lore.kernel.org
 help / color / mirror / Atom feed
From: Parav Pandit via Virtualization <virtualization@lists.linux-foundation.org>
To: Jason Wang <jasowang@redhat.com>
Cc: "tanuj.kamde@amd.com" <tanuj.kamde@amd.com>,
	"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"virtualization@lists.linux-foundation.org"
	<virtualization@lists.linux-foundation.org>,
	"Wu Zongyong" <wuzongyong@linux.alibaba.com>,
	"Si-Wei Liu" <si-wei.liu@oracle.com>,
	"pabloc@xilinx.com" <pabloc@xilinx.com>,
	"Eli Cohen" <elic@nvidia.com>,
	"Zhang Min" <zhang.min9@zte.com.cn>,
	"lulu@redhat.com" <lulu@redhat.com>,
	"Eugenio Pérez" <eperezma@redhat.com>,
	"Piotr.Uminski@intel.com" <Piotr.Uminski@intel.com>,
	"martinh@xilinx.com" <martinh@xilinx.com>,
	"Xie Yongji" <xieyongji@bytedance.com>,
	"dinang@xilinx.com" <dinang@xilinx.com>,
	"habetsm.xilinx@gmail.com" <habetsm.xilinx@gmail.com>,
	Longpeng <longpeng2@huawei.com>,
	"Dan Carpenter" <dan.carpenter@oracle.com>,
	"lvivier@redhat.com" <lvivier@redhat.com>,
	"Christophe JAILLET" <christophe.jaillet@wanadoo.fr>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"ecree.xilinx@gmail.com" <ecree.xilinx@gmail.com>,
	"hanand@xilinx.com" <hanand@xilinx.com>,
	"martinpo@xilinx.com" <martinpo@xilinx.com>,
	"gautam.dawar@amd.com" <gautam.dawar@amd.com>,
	"Zhu Lingshan" <lingshan.zhu@intel.com>
Subject: RE: [PATCH v4 0/4] Implement vdpasim stop operation
Date: Wed, 1 Jun 2022 18:58:50 +0000	[thread overview]
Message-ID: <PH0PR12MB5481CAA3F57892FF7F05B004DCDF9@PH0PR12MB5481.namprd12.prod.outlook.com> (raw)
In-Reply-To: <CACGkMEsSKF_MyLgFdzVROptS3PCcp1y865znLWgnzq9L7CpFVQ@mail.gmail.com>


> From: Jason Wang <jasowang@redhat.com>
> Sent: Tuesday, May 31, 2022 10:42 PM
> 
> Well, the ability to query the virtqueue state was proposed as another
> feature (Eugenio, please correct me). This should be sufficient for making
> virtio-net to be live migrated.
> 
The device is stopped, it won't answer to this special vq config done here.
Programming all of these using cfg registers doesn't scale for on-chip memory and for the speed.

Next would be to program hundreds of statistics of the 64 VQs through giant PCI config space register in some busy polling scheme.

I can clearly see how all these are inefficient for faster LM.
We need an efficient AQ to proceed with at minimum.

> https://lists.oasis-open.org/archives/virtio-comment/202103/msg00008.html
> 
> > Once the device is stopped, its state cannot be queried further as device
> won't respond.
> > It has limited use case.
> > What we need is to stop non admin queue related portion of the device.
_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

WARNING: multiple messages have this Message-ID (diff)
From: Parav Pandit <parav@nvidia.com>
To: Jason Wang <jasowang@redhat.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>,
	"Eugenio Pérez" <eperezma@redhat.com>,
	"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
	"virtualization@lists.linux-foundation.org"
	<virtualization@lists.linux-foundation.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"martinh@xilinx.com" <martinh@xilinx.com>,
	"Stefano Garzarella" <sgarzare@redhat.com>,
	"martinpo@xilinx.com" <martinpo@xilinx.com>,
	"lvivier@redhat.com" <lvivier@redhat.com>,
	"pabloc@xilinx.com" <pabloc@xilinx.com>,
	"Eli Cohen" <elic@nvidia.com>,
	"Dan Carpenter" <dan.carpenter@oracle.com>,
	"Xie Yongji" <xieyongji@bytedance.com>,
	"Christophe JAILLET" <christophe.jaillet@wanadoo.fr>,
	"Zhang Min" <zhang.min9@zte.com.cn>,
	"Wu Zongyong" <wuzongyong@linux.alibaba.com>,
	"lulu@redhat.com" <lulu@redhat.com>,
	"Zhu Lingshan" <lingshan.zhu@intel.com>,
	"Piotr.Uminski@intel.com" <Piotr.Uminski@intel.com>,
	"Si-Wei Liu" <si-wei.liu@oracle.com>,
	"ecree.xilinx@gmail.com" <ecree.xilinx@gmail.com>,
	"gautam.dawar@amd.com" <gautam.dawar@amd.com>,
	"habetsm.xilinx@gmail.com" <habetsm.xilinx@gmail.com>,
	"tanuj.kamde@amd.com" <tanuj.kamde@amd.com>,
	"hanand@xilinx.com" <hanand@xilinx.com>,
	"dinang@xilinx.com" <dinang@xilinx.com>,
	Longpeng <longpeng2@huawei.com>
Subject: RE: [PATCH v4 0/4] Implement vdpasim stop operation
Date: Wed, 1 Jun 2022 18:58:50 +0000	[thread overview]
Message-ID: <PH0PR12MB5481CAA3F57892FF7F05B004DCDF9@PH0PR12MB5481.namprd12.prod.outlook.com> (raw)
In-Reply-To: <CACGkMEsSKF_MyLgFdzVROptS3PCcp1y865znLWgnzq9L7CpFVQ@mail.gmail.com>


> From: Jason Wang <jasowang@redhat.com>
> Sent: Tuesday, May 31, 2022 10:42 PM
> 
> Well, the ability to query the virtqueue state was proposed as another
> feature (Eugenio, please correct me). This should be sufficient for making
> virtio-net to be live migrated.
> 
The device is stopped, it won't answer to this special vq config done here.
Programming all of these using cfg registers doesn't scale for on-chip memory and for the speed.

Next would be to program hundreds of statistics of the 64 VQs through giant PCI config space register in some busy polling scheme.

I can clearly see how all these are inefficient for faster LM.
We need an efficient AQ to proceed with at minimum.

> https://lists.oasis-open.org/archives/virtio-comment/202103/msg00008.html
> 
> > Once the device is stopped, its state cannot be queried further as device
> won't respond.
> > It has limited use case.
> > What we need is to stop non admin queue related portion of the device.

  reply	other threads:[~2022-06-01 18:58 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-26 12:43 [PATCH v4 0/4] Implement vdpasim stop operation Eugenio Pérez
2022-05-26 12:43 ` [PATCH v4 1/4] vdpa: Add " Eugenio Pérez
2022-05-26 14:23   ` Stefano Garzarella
2022-05-26 14:23     ` Stefano Garzarella
2022-05-26 15:32     ` Eugenio Perez Martin
2022-06-01  5:35   ` Eli Cohen
2022-06-01  6:53     ` Eugenio Perez Martin
2022-05-26 12:43 ` [PATCH v4 2/4] vhost-vdpa: introduce STOP backend feature bit Eugenio Pérez
2022-05-26 12:43 ` [PATCH v4 3/4] vhost-vdpa: uAPI to stop the device Eugenio Pérez
2022-06-01 11:03   ` Michael S. Tsirkin
2022-06-01 11:03     ` Michael S. Tsirkin
2022-06-01 11:15     ` Eugenio Perez Martin
2022-06-01 19:13       ` Parav Pandit via Virtualization
2022-06-01 19:13         ` Parav Pandit
2022-06-02  6:21         ` Eugenio Perez Martin
2022-05-26 12:43 ` [PATCH v4 4/4] vdpa_sim: Implement stop vdpa op Eugenio Pérez
2022-05-26 14:25   ` Stefano Garzarella
2022-05-26 14:25     ` Stefano Garzarella
2022-05-26 12:54 ` [PATCH v4 0/4] Implement vdpasim stop operation Parav Pandit
2022-05-26 12:54   ` Parav Pandit via Virtualization
2022-05-27  2:26   ` Jason Wang
2022-05-27  2:26     ` Jason Wang
2022-05-27  7:55     ` Eugenio Perez Martin
2022-05-31 20:26       ` Parav Pandit
2022-05-31 20:26         ` Parav Pandit via Virtualization
2022-06-01 10:48         ` Eugenio Perez Martin
2022-05-27 10:55   ` Michael S. Tsirkin
2022-05-27 10:55     ` Michael S. Tsirkin
2022-05-30  3:39     ` Jason Wang
2022-05-30  3:39       ` Jason Wang
2022-05-31  5:40       ` Michael S. Tsirkin
2022-05-31  5:40         ` Michael S. Tsirkin
2022-05-31  6:44         ` Jason Wang
2022-05-31  6:44           ` Jason Wang
2022-05-31 20:19       ` Parav Pandit
2022-05-31 20:19         ` Parav Pandit via Virtualization
2022-06-01  2:42         ` Jason Wang
2022-06-01  2:42           ` Jason Wang
2022-06-01 18:58           ` Parav Pandit via Virtualization [this message]
2022-06-01 18:58             ` Parav Pandit
2022-06-02  2:00             ` Jason Wang
2022-06-02  2:00               ` Jason Wang
2022-06-02  2:59               ` Parav Pandit
2022-06-02  2:59                 ` Parav Pandit via Virtualization
2022-06-02  3:53                 ` Jason Wang
2022-06-02  3:53                   ` Jason Wang
2022-06-15  0:10                   ` Parav Pandit
2022-06-15  0:10                     ` Parav Pandit via Virtualization
2022-06-15  1:28                     ` Jason Wang
2022-06-15  1:28                       ` Jason Wang
2022-06-16 19:36                       ` Parav Pandit
2022-06-16 19:36                         ` Parav Pandit via Virtualization
2022-06-17  1:15                         ` Jason Wang
2022-06-17  1:15                           ` Jason Wang
2022-06-17  2:42                           ` Parav Pandit
2022-06-17  2:42                             ` Parav Pandit via Virtualization
2022-06-02  8:57                 ` Eugenio Perez Martin
2022-06-01  9:49         ` Eugenio Perez Martin
2022-06-01 19:30           ` Parav Pandit via Virtualization
2022-06-01 19:30             ` Parav Pandit
2022-06-02  2:02             ` Jason Wang
2022-06-02  2:02               ` Jason Wang
2022-05-31  5:42 ` Michael S. Tsirkin
2022-05-31  5:42   ` Michael S. Tsirkin
2022-05-31  7:13   ` Eugenio Perez Martin
2022-05-31  9:23     ` Michael S. Tsirkin
2022-05-31  9:23       ` Michael S. Tsirkin
2022-06-02  2:08 ` Jason Wang
2022-06-02  2:08   ` Jason Wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=PH0PR12MB5481CAA3F57892FF7F05B004DCDF9@PH0PR12MB5481.namprd12.prod.outlook.com \
    --to=virtualization@lists.linux-foundation.org \
    --cc=Piotr.Uminski@intel.com \
    --cc=christophe.jaillet@wanadoo.fr \
    --cc=dan.carpenter@oracle.com \
    --cc=dinang@xilinx.com \
    --cc=ecree.xilinx@gmail.com \
    --cc=elic@nvidia.com \
    --cc=eperezma@redhat.com \
    --cc=gautam.dawar@amd.com \
    --cc=habetsm.xilinx@gmail.com \
    --cc=hanand@xilinx.com \
    --cc=jasowang@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=lingshan.zhu@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=longpeng2@huawei.com \
    --cc=lulu@redhat.com \
    --cc=lvivier@redhat.com \
    --cc=martinh@xilinx.com \
    --cc=martinpo@xilinx.com \
    --cc=mst@redhat.com \
    --cc=netdev@vger.kernel.org \
    --cc=pabloc@xilinx.com \
    --cc=parav@nvidia.com \
    --cc=si-wei.liu@oracle.com \
    --cc=tanuj.kamde@amd.com \
    --cc=wuzongyong@linux.alibaba.com \
    --cc=xieyongji@bytedance.com \
    --cc=zhang.min9@zte.com.cn \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.