Re: [PATCH v2 1/4] vdpa: Add resume operation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

I don't know why but it looks that this series wasn't showing up in the list for some reason. Only the recipients in the To or Cc lines seemed able to get it. Would it be possible to fix the mail client or include me to the thread for review?

Thanks,
-Siwei


On 10/17/2022 12:19 AM, Jason Wang wrote:
On Fri, Oct 14, 2022 at 7:14 PM <sebastien.boeuf@xxxxxxxxx> wrote:
From: Sebastien Boeuf <sebastien.boeuf@xxxxxxxxx>

Add a new operation to allow a vDPA device to be resumed after it has
been suspended. Trying to resume a device that wasn't suspended will
result in a no-op.

This operation is optional. If it's not implemented, the associated
backend feature bit will not be exposed. And if the feature bit is not
exposed, invoking this operation will return an error.

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@xxxxxxxxx>
Acked-by: Jason Wang <jasowang@xxxxxxxxxx>

---
  include/linux/vdpa.h | 6 +++++-
  1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h
index 6d0f5e4e82c2..96d308cbf97b 100644
--- a/include/linux/vdpa.h
+++ b/include/linux/vdpa.h
@@ -219,7 +219,10 @@ struct vdpa_map_file {
   * @reset:                     Reset device
   *                             @vdev: vdpa device
   *                             Returns integer: success (0) or error (< 0)
- * @suspend:                   Suspend or resume the device (optional)
+ * @suspend:                   Suspend the device (optional)
+ *                             @vdev: vdpa device
+ *                             Returns integer: success (0) or error (< 0)
+ * @resume:                    Resume the device (optional)
   *                             @vdev: vdpa device
   *                             Returns integer: success (0) or error (< 0)
   * @get_config_size:           Get the size of the configuration space includes
@@ -324,6 +327,7 @@ struct vdpa_config_ops {
         void (*set_status)(struct vdpa_device *vdev, u8 status);
         int (*reset)(struct vdpa_device *vdev);
         int (*suspend)(struct vdpa_device *vdev);
+       int (*resume)(struct vdpa_device *vdev);
         size_t (*get_config_size)(struct vdpa_device *vdev);
         void (*get_config)(struct vdpa_device *vdev, unsigned int offset,
                            void *buf, unsigned int len);
--
2.34.1

---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris,
92196 Meudon Cedex, France
Registration Number:  302 456 199 R.C.S. NANTERRE
Capital: 5 208 026.16 Euros

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.

_______________________________________________
Virtualization mailing list
Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/virtualization


_______________________________________________
Virtualization mailing list
Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/virtualization



[Index of Archives]     [KVM Development]     [Libvirt Development]     [Libvirt Users]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux