> From: Michael S. Tsirkin <m...@redhat.com> > Sent: Thursday, March 2, 2023 6:47 PM
> > Can you please add other useful error codes in addition to the EINVAL? > > Few that we are needed EAGAIN, ENOMEM, EBUSY, ENODEV. > > We'll add them when there's a use-case. Errno.h is self-explanatory for the use case. A device may want the driver to retry the command as it is busy. > > > And with that, VIRTIO_ADMIN_STATUS_Q_INVALID_MEMBER moves under > ENODEV error code. > > > No I think that belongs in in EINVAL - command is malformed. Command is not malformed. The command prepared is correct, the VF number in the well-formed command which has an invalid VF device number. > ENODEV is more "The wrong type of device was given to a function that > expects a particular sort of device." > Yes. Hence the ENODEV when the VF device is accessed is not valid. --------------------------------------------------------------------- To unsubscribe, e-mail: virtio-dev-unsubscr...@lists.oasis-open.org For additional commands, e-mail: virtio-dev-h...@lists.oasis-open.org
