lkml.org 
[lkml]   [2022]   [Jun]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH] vdpa: allow vdpa dev_del management operation to return failure
Date

> From: Dawar, Gautam <gautam.dawar@amd.com>
> Sent: Wednesday, June 8, 2022 6:30 AM
> To: Jason Wang <jasowang@redhat.com>
> Cc: netdev <netdev@vger.kernel.org>; linux-net-drivers (AMD-Xilinx) <linux-
> net-drivers@amd.com>; Anand, Harpreet <harpreet.anand@amd.com>;
> Michael S. Tsirkin <mst@redhat.com>; Zhu Lingshan
> <lingshan.zhu@intel.com>; Xie Yongji <xieyongji@bytedance.com>; Eli
> Cohen <elic@nvidia.com>; Parav Pandit <parav@nvidia.com>; Si-Wei Liu <si-
> wei.liu@oracle.com>; Stefano Garzarella <sgarzare@redhat.com>; Wan
> Jiabing <wanjiabing@vivo.com>; Dan Carpenter
> <dan.carpenter@oracle.com>; virtualization <virtualization@lists.linux-
> foundation.org>; linux-kernel <linux-kernel@vger.kernel.org>
> Subject: RE: [PATCH] vdpa: allow vdpa dev_del management operation to
> return failure
>
> [AMD Official Use Only - General]
>
> Hi Gautam:
> [GD>>] Hi Jason,
>
> On Fri, Jun 3, 2022 at 6:34 PM Gautam Dawar <gautam.dawar@amd.com>
> wrote:
> >
> > Currently, the vdpa_nl_cmd_dev_del_set_doit() implementation allows
> > returning a value to depict the operation status but the return type
> > of dev_del() callback is void. So, any error while deleting the vdpa
> > device in the vdpa parent driver can't be returned to the management
> > layer.
>
> I wonder under which cognition we can hit an error in dev_del()?
> [GD>>] In the AMD-Xilinx vDPA driver, on receiving vdpa device deletion
> request, I try to identify if the vdpa device is in use by any virtio-net driver
> (through any vdpa bus driver) by looking at the vdpa device status value. In
> case the vdpa device status is >= VIRTIO_CONFIG_S_DRIVER, -EBUSY is
> returned.
> This is to avoid side-effects as noted in
> https://bugzilla.kernel.org/show_bug.cgi?id=213179 caused by deleting the
> vdpa device when it is being used.
> >
User should be able to delete the device anytime.
Upper layers who are unable to perform teardown sequence should be fixed.
\
 
 \ /
  Last update: 2022-06-08 12:44    [W:0.071 / U:0.108 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site