On Wed, Jul 05 2023, Parav Pandit <pa...@nvidia.com> wrote:
> diff --git a/admin-cmds-legacy-interface.tex b/admin-cmds-legacy-interface.tex
> new file mode 100644
> index 0000000..09001d5
> --- /dev/null
> +++ b/admin-cmds-legacy-interface.tex
> @@ -0,0 +1,205 @@
> +\subsubsection{Legacy Interfaces}\label{sec:Basic Facilities of a Virtio
> Device / Device groups / Group
> +administration commands / Legacy Interface}
> +
> +In some systems, there is a need to support utilizing the legacy driver with
> +the device that do not directly support the legacy interface. In such
> scenarios,
Maybe better:
"In some systems, there is a need to support utilizing legacy drivers with
devices that do not directly support the legacy interface."
?
> +a group owner device can provide the legacy interface functionality for the
> +group member devices. The driver of an owner device can then access the
> legacy
> +interface of a member device on behalf of the legacy member device driver.
(...)
> +\paragraph{Legacy Driver Notification Query}\label{par:Basic Facilities of a
> Virtio Device / Device groups / Group administration commands / Legacy
> Interface / Legacy Driver Notifications Query}
> +
> +Even though the driver notifications can be communicated through the
> +administration command, if the group owner device or group member device
> +supports such notifications using a memory-mapped operation or I/O operation,
> +they are sent to the device by accessing such a notification region using
> +a memory or an I/O operation.
I find this sentence confusing, probably because it is too
complex... what about
"If the group owner device or the group member device support driver
notifications via a memory-mapped operation or I/O operation, these
notifications are sent to the device via accessing such a notification
region using a memory or I/O operation {in addition to|instead of}
communicating the notifications through the admistration command."
(I'm not sure whether it is either or both.)
> +
> +A group owner device optionally support querying driver notifications region
"can optionally support" ?
> +using VIRTIO_ADMIN_CMD_LEGACY_NOTIFY_QUERY command.
> +
> +The driver sets command \field{opcode} to
> VIRTIO_ADMIN_CMD_LEGACY_NOTIFY_QUERY.
> +This command does not use \field{command_specific_data}.
> +
> +When command completes successfully, \field{command_specific_result}
"the command"
> +uses following structure:
---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscr...@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-h...@lists.oasis-open.org
