Hi,
Any topics to cover tomorrow? I know Peter has made progress on
virtio-video but is out on holiday this week. I don't think there is
much to report on the rist-vmm work yet but perhaps a quick catch-up?
Anything else?
--
Alex Bennée
This patch adds support for interrupts to the virtio-gpio specification.
This uses the feature bit 0 for the same.
Fixes: https://github.com/oasis-tcs/virtio-spec/issues/114
Cc: Marc Zyngier <maz(a)kernel.org>
Cc: Thomas Gleixner <tglx(a)linutronix.de>
Reviewed-by: Linus Walleij <linus.walleij(a)linaro.org>
Signed-off-by: Viresh Kumar <viresh.kumar(a)linaro.org>
---
V9 -> V10:
- Mandatory for the interrupt to be enabled before being unmasked.
- Remove unnecessary stuff from Device Initialization and clean it up.
- Minor cleanup (s/enabled/negotiated, etc).
V8 -> V9:
- The patch for base GPIO specification is already merged, sending this
separately now.
- Differentiate properly between enabling/disabling and masking/unmasking of the
interrupt.
- Specify how a trigger type should be changed, i.e. by disabling interrupt
first.
- No fixed sequence for enabling/unmasking of the interrupt, any of them can be
done first. The interrupt is only delivered once it is enabled and unmasked.
- Use normative text only in normative sections.
- Guest side Linux driver's IRQ implementation:
https://lore.kernel.org/linux-gpio/96223fb8143a4eaa9b183d376ff46e5cd8ef54b4…
conformance.tex | 2 +
virtio-gpio.tex | 245 +++++++++++++++++++++++++++++++++++++++++++++++-
2 files changed, 246 insertions(+), 1 deletion(-)
diff --git a/conformance.tex b/conformance.tex
index c52f1a40be2d..64bcc12d1199 100644
--- a/conformance.tex
+++ b/conformance.tex
@@ -310,6 +310,7 @@ \section{Conformance Targets}\label{sec:Conformance / Conformance Targets}
\begin{itemize}
\item \ref{drivernormative:Device Types / GPIO Device / requestq Operation}
+\item \ref{drivernormative:Device Types / GPIO Device / eventq Operation}
\end{itemize}
\conformance{\section}{Device Conformance}\label{sec:Conformance / Device Conformance}
@@ -568,6 +569,7 @@ \section{Conformance Targets}\label{sec:Conformance / Conformance Targets}
\begin{itemize}
\item \ref{devicenormative:Device Types / GPIO Device / requestq Operation}
+\item \ref{devicenormative:Device Types / GPIO Device / eventq Operation}
\end{itemize}
\conformance{\section}{Legacy Interface: Transitional Device and Transitional Driver Conformance}\label{sec:Conformance / Legacy Interface: Transitional Device and Transitional Driver Conformance}
diff --git a/virtio-gpio.tex b/virtio-gpio.tex
index 3c614ec97b92..55c553f92a14 100644
--- a/virtio-gpio.tex
+++ b/virtio-gpio.tex
@@ -11,11 +11,17 @@ \subsection{Virtqueues}\label{sec:Device Types / GPIO Device / Virtqueues}
\begin{description}
\item[0] requestq
+\item[1] eventq
\end{description}
+The \field{eventq} virtqueue is available only if the \field{VIRTIO_GPIO_F_IRQ}
+feature is offered by the device.
+
\subsection{Feature bits}\label{sec:Device Types / GPIO Device / Feature bits}
-None currently defined.
+\begin{description}
+\item[VIRTIO_GPIO_F_IRQ (0)] The device supports interrupts on GPIO lines.
+\end{description}
\subsection{Device configuration layout}\label{sec:Device Types / GPIO Device / Device configuration layout}
@@ -46,6 +52,9 @@ \subsection{Device Initialization}\label{sec:Device Types / GPIO Device / Device
\begin{itemize}
\item The driver configures and initializes the \field{requestq} virtqueue.
+
+\item The driver configures and initializes the \field{eventq} virtqueue, if the
+ \field{VIRTIO_GPIO_F_IRQ} feature has been negotiated.
\end{itemize}
\subsection{Device Operation: requestq}\label{sec:Device Types / GPIO Device / requestq Operation}
@@ -105,11 +114,20 @@ \subsection{Device Operation: requestq}\label{sec:Device Types / GPIO Device / r
#define VIRTIO_GPIO_MSG_SET_DIRECTION 0x0003
#define VIRTIO_GPIO_MSG_GET_VALUE 0x0004
#define VIRTIO_GPIO_MSG_SET_VALUE 0x0005
+#define VIRTIO_GPIO_MSG_SET_IRQ_TYPE 0x0006
/* GPIO Direction types */
#define VIRTIO_GPIO_DIRECTION_NONE 0x00
#define VIRTIO_GPIO_DIRECTION_OUT 0x01
#define VIRTIO_GPIO_DIRECTION_IN 0x02
+
+/* GPIO interrupt types */
+#define VIRTIO_GPIO_IRQ_TYPE_NONE 0x00
+#define VIRTIO_GPIO_IRQ_TYPE_EDGE_RISING 0x01
+#define VIRTIO_GPIO_IRQ_TYPE_EDGE_FALLING 0x02
+#define VIRTIO_GPIO_IRQ_TYPE_EDGE_BOTH 0x03
+#define VIRTIO_GPIO_IRQ_TYPE_LEVEL_HIGH 0x04
+#define VIRTIO_GPIO_IRQ_TYPE_LEVEL_LOW 0x08
\end{lstlisting}
\subsubsection{requestq Operation: Get Line Names}\label{sec:Device Types / GPIO Device / requestq Operation / Get Line Names}
@@ -270,6 +288,73 @@ \subsubsection{requestq Operation: Set Value}\label{sec:Device Types / GPIO Devi
\hline
\end{tabularx}
+\subsubsection{requestq Operation: Set IRQ Type}\label{sec:Device Types / GPIO Device / requestq Operation / Set IRQ Type}
+
+This request is allowed only if the \field{VIRTIO_GPIO_F_IRQ} feature has been
+negotiated.
+
+The interrupt configuration is divided into two steps, enabling or disabling of
+the interrupt at the device and masking or unmasking of the interrupt for
+delivery at the driver. This request only pertains to enabling or disabling of
+the interrupt at the device, the masking and unmasking of the interrupt is
+handled by a separate request that takes place over the \field{eventq}
+virtqueue.
+
+The driver sends the \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} message over the
+\field{requestq} virtqueue to enable or disable interrupt for a GPIO line at
+the device.
+
+The driver sends this message with trigger type set to any valid value other
+than \field{VIRTIO_GPIO_IRQ_TYPE_NONE}, to enable the interrupt for a GPIO line,
+this doesn't unmask the interrupt for delivery at the driver though. For edge
+trigger type, the device should latch the interrupt events from this point
+onward and notify it to the driver once the interrupt is unmasked. For level
+trigger type, the device should notify the driver once the interrupt signal on a
+line is sensed and the interrupt is unmasked for the line.
+
+The driver sends this message with trigger type set to
+\field{VIRTIO_GPIO_IRQ_TYPE_NONE}, to disable the interrupt for a GPIO line. The
+device should discard any latched interrupt event associated with it. In order
+to change the trigger type of an already enabled interrupt, the driver should
+first disable the interrupt and then re-enable it with appropriate trigger type.
+
+The interrupts are masked at initialization and the driver unmasks them by
+queuing a pair of buffers, of type \field{virtio_gpio_irq_request} and
+\field{virtio_gpio_irq_response}, over the \field{eventq} virtqueue for a GPIO
+line. A separate pair of buffers must be queued for each GPIO line, the driver
+wants to configure for interrupts. Once an already enabled interrupt is unmasked
+by the driver, the device can notify the driver of an active interrupt signal on
+the GPIO line. This is done by updating the \field{struct
+virtio_gpio_irq_response} buffer's \field{status} with
+\field{VIRTIO_GPIO_IRQ_STATUS_VALID} and returning the updated buffers to the
+driver. The interrupt is masked automatically at this point until the buffers
+are available again at the device.
+
+The interrupt for a GPIO line should not be unmasked before being enabled by the
+driver.
+
+If the interrupt is disabled by the driver, by setting the trigger type to
+\field{VIRTIO_GPIO_IRQ_TYPE_NONE}, or the interrupt is unmasked without being
+enabled first, the device should return any unused pair of buffers for the GPIO
+line, over the \field{eventq} virtqueue, after setting the \field{status} field
+to \field{VIRTIO_GPIO_IRQ_STATUS_INVALID}. This also masks the interrupt.
+
+\begin{tabularx}{\textwidth}{ |l||X|X|X| }
+\hline
+\textbf{Request} & \field{type} & \field{gpio} & \field{value} \\
+\hline
+& \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} & line number & one of \field{VIRTIO_GPIO_IRQ_TYPE_*} \\
+\hline
+\end{tabularx}
+
+\begin{tabularx}{\textwidth}{ |l||X|X| }
+\hline
+\textbf{Response} & \field{status} & \field{value} \\
+\hline
+& \field{VIRTIO_GPIO_STATUS_*} & 0 \\
+\hline
+\end{tabularx}
+
\subsubsection{requestq Operation: Message Flow}\label{sec:Device Types / GPIO Device / requestq Operation / Message Flow}
\begin{itemize}
@@ -313,6 +398,20 @@ \subsubsection{requestq Operation: Message Flow}\label{sec:Device Types / GPIO D
\item The driver MAY send multiple messages for same or different GPIO lines in
parallel.
+
+\item The driver MUST NOT send IRQ messages if the \field{VIRTIO_GPIO_F_IRQ}
+ feature has not been negotiated.
+
+\item The driver MUST NOT send IRQ messages for a GPIO line configured for
+ output.
+
+\item The driver MUST set the IRQ trigger type to
+ \field{VIRTIO_GPIO_IRQ_TYPE_NONE} once it is done using the GPIO line
+ configured for interrupts.
+
+\item In order to change the trigger type of an already enabled interrupt, the
+ driver MUST first disable the interrupt and then re-enable it with
+ appropriate trigger type.
\end{itemize}
\devicenormative{\subsubsection}{requestq Operation}{Device Types / GPIO Device / requestq Operation}
@@ -344,4 +443,148 @@ \subsubsection{requestq Operation: Message Flow}\label{sec:Device Types / GPIO D
\item The device MUST discard all state information corresponding to a GPIO
line, once the driver has requested to set its direction to
\field{VIRTIO_GPIO_DIRECTION_NONE}.
+
+\item The device MUST latch an edge interrupt if the interrupt is enabled but
+ still masked.
+
+\item The device MUST NOT latch an level interrupt if the interrupt is enabled
+ but still masked.
+
+\item The device MUST discard any latched interrupt for a GPIO line, once
+ interrupt is disabled for the same.
+\end{itemize}
+
+\subsection{Device Operation: eventq}\label{sec:Device Types / GPIO Device / eventq Operation}
+
+The \field{eventq} virtqueue is used by the driver to unmask the interrupts and
+used by the device to notify the driver of newly sensed interrupts. In order to
+unmask interrupt on a GPIO line, the driver queues a pair of buffers,
+\field{struct virtio_gpio_irq_request} (filled by driver) and \field{struct
+virtio_gpio_irq_response} (to be filled by device later), to the \field{eventq}
+virtqueue. A separate pair of buffers must be queued for each GPIO line, the
+driver wants to configure for interrupts. The device, on sensing an interrupt,
+returns the pair of buffers for the respective GPIO line, which also masks the
+interrupts. The driver can queue the buffers again to unmask the interrupt.
+
+\begin{lstlisting}
+struct virtio_gpio_irq_request {
+ le16 gpio;
+};
+\end{lstlisting}
+
+This structure is filled by the driver and read by the device.
+
+\begin{description}
+\item[\field{gpio}] is the GPIO line number, i.e. 0 <= \field{gpio} <
+ \field{ngpio}.
+\end{description}
+
+\begin{lstlisting}
+struct virtio_gpio_irq_response {
+ u8 status;
+};
+
+/* Possible values of the interrupt status field */
+#define VIRTIO_GPIO_IRQ_STATUS_INVALID 0x0
+#define VIRTIO_GPIO_IRQ_STATUS_VALID 0x1
+\end{lstlisting}
+
+This structure is filled by the device and read by the driver.
+
+\begin{description}
+\item[\field{status}] of the interrupt event,
+ \field{VIRTIO_GPIO_IRQ_STATUS_VALID} on interrupt and
+ \field{VIRTIO_GPIO_IRQ_STATUS_INVALID} to return the buffers back to the
+ driver after interrupt is disabled.
+\end{description}
+
+\subsubsection{eventq Operation: Message Flow}\label{sec:Device Types / GPIO Device / eventq Operation / Message Flow}
+
+\begin{itemize}
+\item The virtio-gpio driver is requested by a client driver to enable interrupt
+ for a GPIO line and configure it to a particular trigger type.
+
+\item The driver sends the \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} message, over
+ the \field{requestq} virtqueue, and the device configures the GPIO line for
+ the requested trigger type and enables the interrupt. The interrupt is still
+ masked for delivery though. The device shall latch the interrupt from now
+ onward for edge trigger type.
+
+\item The driver unmasks the interrupt by queuing a pair of buffers to the
+ \field{eventq} virtqueue for the GPIO line. The driver can do this before
+ enabling the interrupt as well, though the interrupt must be both unmasked
+ and enabled to get delivered at the driver.
+
+\item The driver notifies the device of the presence of new buffers on the
+ \field{eventq} virtqueue. The interrupt is fully configured at this point.
+
+\item The device, on sensing an active interrupt on the GPIO line, finds the
+ matching buffers (based on GPIO line number) from the \field{eventq}
+ virtqueue and update its \field{struct virtio_gpio_irq_response} buffer's
+ \field{status} with \field{VIRTIO_GPIO_IRQ_STATUS_VALID} and returns the
+ pair of buffers to the device. This results in masking the interrupt as
+ well.
+
+\item The device notifies the driver of the presence of returned buffers on the
+ \field{eventq} virtqueue.
+
+\item If the GPIO line is configured for level interrupts, the device ignores
+ any further interrupt signals on this GPIO line, until the interrupt is
+ unmasked again by the driver by making the buffers available to the device.
+ Once the interrupt is unmasked again and the interrupt on the line is still
+ active, the device shall notify the driver again.
+
+\item If the GPIO line is configured for edge interrupts, the device latches
+ the interrupt received for this GPIO line, until the interrupt is unmasked
+ again by the driver by making the buffers available to the device. Once the
+ interrupt is unmasked again and an interrupt was latched earlier, the
+ device shall notify the driver again.
+
+\item The driver on receiving the notification from the device, processes the
+ interrupt. The interrupt is masked at the device until the buffers are
+ queued again by the driver.
+
+\item In a typical guest operating system kernel, the virtio-gpio driver
+ notifies the client driver, that is associated with this GPIO line, to
+ process the event. In the case of a level triggered interrupt, the client
+ driver shall fully process and acknowledge the event at its source to return
+ the line to its inactive state before the interrupt is unmasked again to
+ avoid a spurious interrupt.
+
+\item Once the interrupt is handled, the driver may queue a pair of buffers for
+ the GPIO line to unmask the interrupt again.
+
+\item The driver can also disable the interrupt by sending the
+ \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} message, with
+ \field{VIRTIO_GPIO_IRQ_TYPE_NONE} trigger type. In that case, the device
+ shall return the unused pair of buffers for the GPIO line after setting the
+ \field{status} field with \field{VIRTIO_GPIO_IRQ_STATUS_INVALID}.
+\end{itemize}
+
+\drivernormative{\subsubsection}{eventq Operation}{Device Types / GPIO Device / eventq Operation}
+
+\begin{itemize}
+\item The driver MUST both enable and unmask the interrupt in order to get
+ notified for the same.
+
+\item The driver MUST enable the interrupt before unmasking it.
+
+\item To unmask the interrupt, the driver MUST queue a separate pair of buffers
+ to the \field{eventq} virtqueue for each GPIO line.
+
+\item The driver MUST NOT add multiple pairs of buffers for the same GPIO line
+ on the \field{eventq} virtqueue.
+\end{itemize}
+
+\devicenormative{\subsubsection}{eventq Operation}{Device Types / GPIO Device / eventq Operation}
+
+\begin{itemize}
+\item The device MUST NOT send an interrupt event to the driver for a GPIO
+ line unless the interrupt has been both unmasked and enabled by the
+ driver.
+
+\item On receiving \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} message, with
+ \field{VIRTIO_GPIO_IRQ_TYPE_NONE} trigger type, the device MUST return the
+ buffers, if they were received earlier, after setting the \field{status}
+ field to \field{VIRTIO_GPIO_IRQ_STATUS_INVALID}.
\end{itemize}
--
2.31.1.272.g89b43f80a514
Hi,
I've dumped a bunch of files on my Stratos file share:
https://fileserver.linaro.org/s/jC3Xq2q297j5bQ8
The first directory (xenpi4.boot) is the result of a Yocto demo build
(as per Christopher Clark's instructions) and contains a known good
version of the firmware, u-boot and with the rest of the image would
boot a Yocto distro image.
As I like understand the full process and I wanted to have a full self
hosted distro set up for development I set up a TFTP based setup where
u-boot gets the hypervisor, kernel and DTB over the network. Most of the
work it turned out was in finding the correct u-boot runes to both
subtly modify the DTB to account for HW variations (for some reason I
couldn't get u-boot to modify the master DTB from the GPU firmware). We
also need to make some modifications to /chosen nodes to ensure Xen
boots up out DomO kernel.
The source of the script (ajb.configs/boot-xenpi.source) is a heavily
commented boot script that will give you the blow-by-blow details. With
this I can boot my own custom hypervisor and kernel into a Debian
Bullseye install. You can find known working configs for those
components in the directory as well.
My distro is running a xen-upstream.deb built from the Xen source tree:
make -j9 debball CROSS_COMPILE=aarch64-linux-gnu- XEN_TARGET_ARCH=arm64
I've also hacked the distro a little:
- removed the kernels and raspi-firmware packages (so as not to mess with /boot/firmware)
- don' auto-mount /boot.firmware
- hand built a QEMU which is pointed to by QEMU_XEN in /etc/default/xencommons
Be aware the Dom0 has 1G of the RAM by default, if you plan to do some
heavy building you might want to give it a bit more memory to play with.
I'll upload my test image and xen config in due course but this should
give something that is fairly easy to iterate with. I've not yet run
rustup on the pi itself though ;-)
--
Alex Bennée
Hi Viresh/Mathieu,
This is a dump of the current state of my notes for cross-building and
running Xen via QEMU. It's a little bit of a stream of consciousness as
I wrote things down but hopefully it proves useful in getting things
started.
━━━━━━━━━━━━━━━━━━━
XEN RELATED NOTES
Alex Bennée
━━━━━━━━━━━━━━━━━━━
Table of Contents
─────────────────
1. Tasks [0/1] :tasks:
.. 1. TODO Xen SMC/HVC pass through ([STR-41])
2. Notes
.. 1. Building Xen
.. 2. Cross Build
.. 3. Cross Packages
.. 4. Configure
.. 5. Build
..... 1. Just the Hypervisor
.. 6. Running Dom0
.. 7. MachiatoBin Issues
.. 8. Running DomU
.. 9. View console of DomU guest
1 Tasks [0/1] :tasks:
═════════════
[STR-41] <https://projects.linaro.org/browse/STR-41>
1.1 TODO Xen SMC/HVC pass through ([STR-41])
────────────────────────────────────────────
> sstabellini_: that tracks [ 1.420679] mvebu-comphy f2120000.phy:
> unsupported SMC call, try updating your firmware [ 1.420720]
> mvebu-comphy f2120000.phy: Firmware could not configure PHY 4 with
> mode 15 (ret: -95), trying legacy method sstabellini_: is that a
> boot option? <sstabellini_> stsquad: as an example see [19:55]
> xen/arch/arm/platforms/xilinx-zynqmp-eemi.c:forward_to_fw
> <sstabellini_> stsquad: we don't have a boot option for that, you
> need to add a few lines of code. Actually this issue comes up often
> enough that a boot option would be really useful!
[STR-41] <https://projects.linaro.org/browse/STR-41>
2 Notes
═══════
2.1 Building Xen
────────────────
As Xen's out-of-tree support is a little sporadic I've ended up going
for using git-worktree to split out the various builds.
2.2 Cross Build
───────────────
2.3 Cross Packages
──────────────────
┌────
│ apt install libpython3-dev:arm64 libfdt-dev:arm64 libncurses-dev:arm64
└────
Or as I do on hackbox do everything in a container:
┌────
│ docker run --rm -it -u (id -u) -v $HOME:$HOME -w (pwd) alex.bennee:xen-arm64 /usr/bin/fish
└────
built from [my dockerfile]
[my dockerfile]
<https://github.com/stsquad/dockerfiles/blob/master/crossbuild/xen-arm64/Doc…>
2.4 Configure
─────────────
┌────
│ ./configure --build=x86_64-unknown-linux-gnu --host=aarch64-linux-gnu \
│ --disable-docs --disable-golang --disable-ocamltools \
│ --with-system-qemu=/usr/bin/qemu-system-i386
└────
The location of system QEMU can be tweaked after the fact by editing
/etc/default/xencommons
2.5 Build
─────────
You need to have the compilers in the command line:
┌────
│ make -j9 dist CROSS_COMPILE=aarch64-linux-gnu- XEN_TARGET_ARCH=arm64
└────
To build an Debian package which can install all the tools and
binaries (replacing any distro requirements) then:
┌────
│ make -j9 debball CROSS_COMPILE=aarch64-linux-gnu- XEN_TARGET_ARCH=arm64
└────
This builds a package which you can find in
./dist/xen-upstream-4.15-unstable.deb which you can dpkg -i on your
dom0 environment.
2.5.1 Just the Hypervisor
╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌
┌────
│ cd xen
│ set -x CROSS_COMPILE aarch64-linux-gnu-
│ make XEN_TARGET_ARCH=arm64
└────
2.6 Running Dom0
────────────────
To avoid complications with broken grub and bios firmware I'm
currently just using my direct boot:
┌────
│ ./qemu-system-aarch64 -machine virt,virtualization=on \
│ -cpu cortex-a57 -serial mon:stdio \
│ -nic user,model=virtio-net-pci,hostfwd=tcp::2222-:22 \
│ -device virtio-scsi-pci \
│ -drive file=/dev/zvol/hackpool-0/debian-buster-arm64,id=hd0,index=0,if=none,format=raw \
│ -device scsi-hd,drive=hd0 \
│ -display none \
│ -m 8192 \
│ -kernel ~/lsrc/xen/xen.build.arm64-xen/xen/xen \
│ -append "dom0_mem=2G,max:2G dom0_max_vcpus=4 loglvl=all guest_loglvl=all" \
│ -device guest-loader,addr=0x46000000,kernel=$HOME/lsrc/linux.git/builds/arm64/arch/arm64/boot/Image,bootargs="root=/dev/sda2 console=hvc0 earlyprintk=xen" \
│ -smp 8
└────
Care has to be taken to avoid the guest-loader address clashing with
anything else and ending up corrupting the DTB. Currently QEMU doesn't
do anything avoid those issues as it doesn't have visibility of where
the kernel gets loaded. You can see by examining the Xen output:
┌────
│ (XEN) Loading zImage from 0000000046000000 to 0000000050000000-0000000050eb2200
│ (XEN) Loading d0 DTB to 0x0000000058000000-0x0000000058001ce8
└────
The initial xl list will check Xen and it's tools are up and running
and the hypervisor and userspace ABI's are in sync. It's important
that the hypercall ABI and it's userspace are in sync.
┌────
│ 13:13:43 [root@buster:~] # xl list
│ Name ID Mem VCPUs State Time(s)
│ Domain-0 0 4096 4 r----- 215.8
└────
2.7 MachiatoBin Issues
──────────────────────
• upgraded to Bullseye for latest Grub
• built with 8250 serial (32 bit, shift = 2)
2.8 Running DomU
────────────────
The with a config like
┌────
│ # =====================================================================# Example PV Linux guest configuration
│ # =====================================================================
│ #
│ # This is a fairly minimal example of what is required for a
│ # Paravirtualised Linux guest. For a more complete guide see xl.cfg(5)
│
│ # Guest name
│ name = "xenpv-initrd-guest"
│
│ # 128-bit UUID for the domain as a hexadecimal number.
│ # Use "uuidgen" to generate one if required.
│ # The default behavior is to generate a new UUID each time the guest is started.
│ #uuid = "XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX"
│
│ # Kernel image to boot
│ kernel = "/home/alex/arm64-minkern"
│ # kernel = "/home/alex/arm64-kernel"
│ # kernel = "/home/alex/arm64-defconfig"
│
│ # Ramdisk (optional)
│ #ramdisk = "/boot/initrd.gz"
│
│ # Kernel command line options
│ extra = "console=hvc0"
│
│ # Initial memory allocation (MB)
│ memory = 4096
│
│ # Maximum memory (MB)
│ # If this is greater than `memory' then the slack will start ballooned
│ # (this assumes guest kernel support for ballooning)
│ maxmem = 4096
│
│ # Number of VCPUS
│ vcpus = 2
│
│ # Network devices
│ # A list of 'vifspec' entries as described in
│ # docs/misc/xl-network-configuration.markdown
│ # vif = [ ]
│
│ # Disk Devices
│ # A list of `diskspec' entries as described in
│ # docs/misc/xl-disk-configuration.txt
│ # disk = [ '/dev/vg/guest-volume,raw,xvda,rw' ]
│ # disk = [ ]
└────
┌────
│ xl create simple-guest.conf
│ xl list
│ Name ID Mem VCPUs State Time(s)
│ Domain-0 0 4096 4 r----- 221.4
│ xenpv-initrd-guest 1 4095 2 -b---- 5.3
└────
2.9 View console of DomU guest
──────────────────────────────
And to get console:
┌────
│ xl console -t pv -n 0 xenpv-initrd-guest
└────
to exit you need *Ctrl-[*
--
Alex Bennée
This patch adds support for interrupts to the virtio-gpio specification.
This uses the feature bit 0 for the same.
Fixes: https://github.com/oasis-tcs/virtio-spec/issues/114
Cc: Marc Zyngier <maz(a)kernel.org>
Cc: Thomas Gleixner <tglx(a)linutronix.de>
Reviewed-by: Linus Walleij <linus.walleij(a)linaro.org>
Signed-off-by: Viresh Kumar <viresh.kumar(a)linaro.org>
---
V8 -> V9:
- The patch for base GPIO specification is already merged, sending this
separately now.
- Differentiate properly between enabling/disabling and masking/unmasking of the
interrupt.
- Specify how a trigger type should be changed, i.e. by disabling interrupt
first.
- No fixed sequence for enabling/unmasking of the interrupt, any of them can be
done first. The interrupt is only delivered once it is enabled and unmasked.
- Use normative text only in normative sections.
- Guest side Linux driver's IRQ implementation:
https://lore.kernel.org/linux-gpio/96223fb8143a4eaa9b183d376ff46e5cd8ef54b4…
conformance.tex | 2 +
virtio-gpio.tex | 248 +++++++++++++++++++++++++++++++++++++++++++++++-
2 files changed, 249 insertions(+), 1 deletion(-)
diff --git a/conformance.tex b/conformance.tex
index c52f1a40be2d..64bcc12d1199 100644
--- a/conformance.tex
+++ b/conformance.tex
@@ -310,6 +310,7 @@ \section{Conformance Targets}\label{sec:Conformance / Conformance Targets}
\begin{itemize}
\item \ref{drivernormative:Device Types / GPIO Device / requestq Operation}
+\item \ref{drivernormative:Device Types / GPIO Device / eventq Operation}
\end{itemize}
\conformance{\section}{Device Conformance}\label{sec:Conformance / Device Conformance}
@@ -568,6 +569,7 @@ \section{Conformance Targets}\label{sec:Conformance / Conformance Targets}
\begin{itemize}
\item \ref{devicenormative:Device Types / GPIO Device / requestq Operation}
+\item \ref{devicenormative:Device Types / GPIO Device / eventq Operation}
\end{itemize}
\conformance{\section}{Legacy Interface: Transitional Device and Transitional Driver Conformance}\label{sec:Conformance / Legacy Interface: Transitional Device and Transitional Driver Conformance}
diff --git a/virtio-gpio.tex b/virtio-gpio.tex
index 3c614ec97b92..fba7ec400bb3 100644
--- a/virtio-gpio.tex
+++ b/virtio-gpio.tex
@@ -11,11 +11,17 @@ \subsection{Virtqueues}\label{sec:Device Types / GPIO Device / Virtqueues}
\begin{description}
\item[0] requestq
+\item[1] eventq
\end{description}
+The \field{eventq} virtqueue is available only if the \field{VIRTIO_GPIO_F_IRQ}
+feature is enabled by the device.
+
\subsection{Feature bits}\label{sec:Device Types / GPIO Device / Feature bits}
-None currently defined.
+\begin{description}
+\item[VIRTIO_GPIO_F_IRQ (0)] The device supports interrupts on GPIO lines.
+\end{description}
\subsection{Device configuration layout}\label{sec:Device Types / GPIO Device / Device configuration layout}
@@ -46,6 +52,14 @@ \subsection{Device Initialization}\label{sec:Device Types / GPIO Device / Device
\begin{itemize}
\item The driver configures and initializes the \field{requestq} virtqueue.
+
+\item The driver checks the presence of \field{VIRTIO_GPIO_F_IRQ} feature
+ before initiating any IRQ related messages.
+
+\item The driver configures and initializes the \field{eventq} virtqueue.
+
+\item The device configures all GPIO lines in \field{VIRTIO_GPIO_IRQ_TYPE_NONE}
+ trigger type state.
\end{itemize}
\subsection{Device Operation: requestq}\label{sec:Device Types / GPIO Device / requestq Operation}
@@ -105,11 +119,20 @@ \subsection{Device Operation: requestq}\label{sec:Device Types / GPIO Device / r
#define VIRTIO_GPIO_MSG_SET_DIRECTION 0x0003
#define VIRTIO_GPIO_MSG_GET_VALUE 0x0004
#define VIRTIO_GPIO_MSG_SET_VALUE 0x0005
+#define VIRTIO_GPIO_MSG_SET_IRQ_TYPE 0x0006
/* GPIO Direction types */
#define VIRTIO_GPIO_DIRECTION_NONE 0x00
#define VIRTIO_GPIO_DIRECTION_OUT 0x01
#define VIRTIO_GPIO_DIRECTION_IN 0x02
+
+/* GPIO interrupt types */
+#define VIRTIO_GPIO_IRQ_TYPE_NONE 0x00
+#define VIRTIO_GPIO_IRQ_TYPE_EDGE_RISING 0x01
+#define VIRTIO_GPIO_IRQ_TYPE_EDGE_FALLING 0x02
+#define VIRTIO_GPIO_IRQ_TYPE_EDGE_BOTH 0x03
+#define VIRTIO_GPIO_IRQ_TYPE_LEVEL_HIGH 0x04
+#define VIRTIO_GPIO_IRQ_TYPE_LEVEL_LOW 0x08
\end{lstlisting}
\subsubsection{requestq Operation: Get Line Names}\label{sec:Device Types / GPIO Device / requestq Operation / Get Line Names}
@@ -270,6 +293,74 @@ \subsubsection{requestq Operation: Set Value}\label{sec:Device Types / GPIO Devi
\hline
\end{tabularx}
+\subsubsection{requestq Operation: Set IRQ Type}\label{sec:Device Types / GPIO Device / requestq Operation / Set IRQ Type}
+
+This request is allowed only if the \field{VIRTIO_GPIO_F_IRQ} feature is enabled
+by the device.
+
+The interrupt configuration is divided into two steps, enabling or disabling of
+the interrupt at the device and masking or unmasking of the interrupt for
+delivery at the driver. This request only pertains to enabling or disabling of
+the interrupt at the device, the masking and unmasking of the interrupt is
+handled by a separate request that takes place over the \field{eventq}
+virtqueue.
+
+The driver sends the \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} message over the
+\field{requestq} virtqueue to enable or disable interrupt for a GPIO line at
+the device.
+
+The driver sends this message with trigger type set to any valid value other
+than \field{VIRTIO_GPIO_IRQ_TYPE_NONE}, to enable the interrupt for a GPIO line,
+this doesn't unmask the interrupt for delivery at the driver though. For edge
+trigger type, the device should latch the interrupt events from this point
+onward and notify it to the driver once the interrupt is unmasked. For level
+trigger type, the device should notify the driver once the interrupt signal on a
+line is sensed and the interrupt is unmasked for the line.
+
+The driver sends this message with trigger type set to
+\field{VIRTIO_GPIO_IRQ_TYPE_NONE}, to disable the interrupt for a GPIO line. The
+device should discard any latched interrupt event associated with it. In order
+to change the trigger type of an already enabled interrupt, the driver should
+first disable the interrupt and then re-enable it with appropriate trigger type.
+
+The interrupts are masked at initialization and the driver unmasks them by
+queuing a pair of buffers, of type \field{virtio_gpio_irq_request} and
+\field{virtio_gpio_irq_response}, over the \field{eventq} virtqueue for a GPIO
+line. A separate pair of buffers must be queued for each GPIO line, the driver
+wants to configure for interrupts. Once the interrupt is unmasked by the driver
+and the interrupt is also enabled at the device, the device can notify the
+driver of an active interrupt signal on the GPIO line. This is done by updating
+the \field{struct virtio_gpio_irq_response} buffer's \field{status} with
+\field{VIRTIO_GPIO_IRQ_STATUS_VALID} and returning the updated buffers to the
+driver. The interrupt is masked automatically at this point until the buffers
+are available again at the device.
+
+When the interrupt is disabled by the driver, by setting the trigger type to
+\field{VIRTIO_GPIO_IRQ_TYPE_NONE}, the device should return any unused pair of
+buffers for the GPIO line, over the \field{eventq} virtqueue, after setting the
+\field{status} field to \field{VIRTIO_GPIO_IRQ_STATUS_INVALID}. This also masks
+the interrupt.
+
+The driver can enable and unmask the interrupt in any order, i.e. it can enable
+the interrupt first and then queue the buffers or queue the buffers first and
+then enable the interrupt.
+
+\begin{tabularx}{\textwidth}{ |l||X|X|X| }
+\hline
+\textbf{Request} & \field{type} & \field{gpio} & \field{value} \\
+\hline
+& \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} & line number & one of \field{VIRTIO_GPIO_IRQ_TYPE_*} \\
+\hline
+\end{tabularx}
+
+\begin{tabularx}{\textwidth}{ |l||X|X| }
+\hline
+\textbf{Response} & \field{status} & \field{value} \\
+\hline
+& \field{VIRTIO_GPIO_STATUS_*} & 0 \\
+\hline
+\end{tabularx}
+
\subsubsection{requestq Operation: Message Flow}\label{sec:Device Types / GPIO Device / requestq Operation / Message Flow}
\begin{itemize}
@@ -313,6 +404,20 @@ \subsubsection{requestq Operation: Message Flow}\label{sec:Device Types / GPIO D
\item The driver MAY send multiple messages for same or different GPIO lines in
parallel.
+
+\item The driver MUST NOT send IRQ messages if the \field{VIRTIO_GPIO_F_IRQ}
+ feature is not enabled by the device.
+
+\item The driver MUST NOT send IRQ messages for a GPIO line configured for
+ output.
+
+\item The driver MUST set the IRQ trigger type to
+ \field{VIRTIO_GPIO_IRQ_TYPE_NONE} once it is done using the GPIO line
+ configured for interrupts.
+
+\item In order to change the trigger type of an already enabled interrupt, the
+ driver MUST first disable the interrupt and then re-enable it with
+ appropriate trigger type.
\end{itemize}
\devicenormative{\subsubsection}{requestq Operation}{Device Types / GPIO Device / requestq Operation}
@@ -344,4 +449,145 @@ \subsubsection{requestq Operation: Message Flow}\label{sec:Device Types / GPIO D
\item The device MUST discard all state information corresponding to a GPIO
line, once the driver has requested to set its direction to
\field{VIRTIO_GPIO_DIRECTION_NONE}.
+
+\item The device MUST latch an edge interrupt if the interrupt is enabled but
+ still masked.
+
+\item The device MUST NOT latch an level interrupt if the interrupt is enabled
+ but still masked.
+
+\item The device MUST discard any latched interrupt for a GPIO line, once
+ interrupt is disabled for the same.
+\end{itemize}
+
+\subsection{Device Operation: eventq}\label{sec:Device Types / GPIO Device / eventq Operation}
+
+The \field{eventq} virtqueue is used by the driver to unmask the interrupts and
+used by the device to notify the driver of newly sensed interrupts. In order to
+unmask interrupt on a GPIO line, the driver queues a pair of buffers,
+\field{struct virtio_gpio_irq_request} (filled by driver) and \field{struct
+virtio_gpio_irq_response} (to be filled by device later), to the \field{eventq}
+virtqueue. A separate pair of buffers must be queued for each GPIO line, the
+driver wants to configure for interrupts. The device, on sensing an interrupt,
+returns the pair of buffers for the respective GPIO line, which also masks the
+interrupts. The driver can queue the buffers again to unmask the interrupt.
+
+\begin{lstlisting}
+struct virtio_gpio_irq_request {
+ le16 gpio;
+};
+\end{lstlisting}
+
+This structure is filled by the driver and read by the device.
+
+\begin{description}
+\item[\field{gpio}] is the GPIO line number, i.e. 0 <= \field{gpio} <
+ \field{ngpio}.
+\end{description}
+
+\begin{lstlisting}
+struct virtio_gpio_irq_response {
+ u8 status;
+};
+
+/* Possible values of the interrupt status field */
+#define VIRTIO_GPIO_IRQ_STATUS_INVALID 0x0
+#define VIRTIO_GPIO_IRQ_STATUS_VALID 0x1
+\end{lstlisting}
+
+This structure is filled by the device and read by the driver.
+
+\begin{description}
+\item[\field{status}] of the interrupt event,
+ \field{VIRTIO_GPIO_IRQ_STATUS_VALID} on interrupt and
+ \field{VIRTIO_GPIO_IRQ_STATUS_INVALID} to return the buffers back to the
+ driver after interrupt is disabled.
+\end{description}
+
+\subsubsection{eventq Operation: Message Flow}\label{sec:Device Types / GPIO Device / eventq Operation / Message Flow}
+
+\begin{itemize}
+\item The virtio-gpio driver is requested by a client driver to enable interrupt
+ for a GPIO line and configure it to a particular trigger type.
+
+\item The driver sends the \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} message, over
+ the \field{requestq} virtqueue, and the device configures the GPIO line for
+ the requested trigger type and enables the interrupt. The interrupt is still
+ masked for delivery though. The device shall latch the interrupt from now
+ onward for edge trigger type.
+
+\item The driver unmasks the interrupt by queuing a pair of buffers to the
+ \field{eventq} virtqueue for the GPIO line. The driver can do this before
+ enabling the interrupt as well, though the interrupt must be both unmasked
+ and enabled to get delivered at the driver.
+
+\item The driver notifies the device of the presence of new buffers on the
+ \field{eventq} virtqueue. The interrupt is fully configured at this point.
+
+\item The device, on sensing an active interrupt on the GPIO line, finds the
+ matching buffers (based on GPIO line number) from the \field{eventq}
+ virtqueue and update its \field{struct virtio_gpio_irq_response} buffer's
+ \field{status} with \field{VIRTIO_GPIO_IRQ_STATUS_VALID} and returns the
+ pair of buffers to the device. This results in masking the interrupt as
+ well.
+
+\item The device notifies the driver of the presence of returned buffers on the
+ \field{eventq} virtqueue.
+
+\item If the GPIO line is configured for level interrupts, the device ignores
+ any further interrupt signals on this GPIO line, until the interrupt is
+ unmasked again by the driver by making the buffers available to the device.
+ Once the interrupt is unmasked again and the interrupt on the line is still
+ active, the device shall notify the driver again.
+
+\item If the GPIO line is configured for edge interrupts, the device latches
+ the interrupt received for this GPIO line, until the interrupt is unmasked
+ again by the driver by making the buffers available to the device. Once the
+ interrupt is unmasked again and an interrupt was latched earlier, the
+ device shall notify the driver again.
+
+\item The driver on receiving the notification from the device, processes the
+ interrupt. The interrupt is masked at the device until the buffers are
+ queued again by the driver.
+
+\item In a typical guest operating system kernel, the virtio-gpio driver
+ notifies the client driver, that is associated with this GPIO line, to
+ process the event. In the case of a level triggered interrupt, the client
+ driver shall fully process and acknowledge the event at its source to return
+ the line to its inactive state before the interrupt is unmasked again to
+ avoid a spurious interrupt.
+
+\item Once the interrupt is handled, the driver may queue a pair of buffers for
+ the GPIO line to unmask the interrupt again.
+
+\item The driver can also disable the interrupt by sending the
+ \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} message, with
+ \field{VIRTIO_GPIO_IRQ_TYPE_NONE} trigger type. In that case, the device
+ shall return the unused pair of buffers for the GPIO line after setting the
+ \field{status} field with \field{VIRTIO_GPIO_IRQ_STATUS_INVALID}.
+\end{itemize}
+
+\drivernormative{\subsubsection}{eventq Operation}{Device Types / GPIO Device / eventq Operation}
+
+\begin{itemize}
+\item The driver MUST both enable and unmask the interrupt in order to get
+ notified for the same.
+
+\item To unmask the interrupt, the driver MUST queue a separate pair of buffers
+ to the \field{eventq} virtqueue for each GPIO line.
+
+\item The driver MUST NOT add multiple pairs of buffers for the same GPIO line
+ on the \field{eventq} virtqueue.
+\end{itemize}
+
+\devicenormative{\subsubsection}{eventq Operation}{Device Types / GPIO Device / eventq Operation}
+
+\begin{itemize}
+\item The device CAN ONLY send an interrupt event to the driver for a GPIO line,
+ if the interrupt is both unmasked and enabled by the driver.
+
+\item On receiving \field{VIRTIO_GPIO_MSG_SET_IRQ_TYPE} message, with
+ \field{VIRTIO_GPIO_IRQ_TYPE_NONE} trigger type, the device MUST return the
+ buffers, if they were received earlier, after setting the \field{status}
+ field to \field{VIRTIO_GPIO_IRQ_STATUS_INVALID}.
\end{itemize}
--
2.31.1.272.g89b43f80a514
Hi,
Arnd suggested (over IRC) to split this into two patches for better readability
and so here is a resend. The eventual specification hasn't changed at all.
V4->V5:
- Split into two patches.
V3->V4:
- Add a new mandatory feature flag.
V2->V3:
- Add conformance clauses that require that the flag is consistent with the
buffer.
V1->V2:
- Name the buffer-less request as zero-length request.
------
I did try to follow the discussion you guys had during V4, where we added
support for multiple buffers for the same request, which I think is unnecessary
now, after introduction of the VIRTIO_I2C_FLAGS_FAIL_NEXT flag.
https://lists.oasis-open.org/archives/virtio-comment/202011/msg00005.html
And so starting this discussion again, because we need to support stuff
like: i2cdetect -q <i2c-bus-number>, which issues a zero-length SMBus
Quick command.
Viresh Kumar (2):
virtio: i2c: No need to have separate read-write buffers
virtio: i2c: Allow zero-length transactions
virtio-i2c.tex | 76 ++++++++++++++++++++++++++++++--------------------
1 file changed, 45 insertions(+), 31 deletions(-)
--
2.31.1.272.g89b43f80a514
Hi,
This patchset simplifies the protocol and allows zero-length transactions, which
are required to support stuff like: i2cdetect -q <i2c-bus-number>, which issues
a zero-length SMBus Quick command.
V5->V6:
- s/SMBus Quick/the SMBus "Quick" command/
- Add a footnote and reword/rearrange few parts for more clarity.
V4->V5:
- Split into two patches.
V3->V4:
- Add a new mandatory feature flag.
V2->V3:
- Add conformance clauses that require that the flag is consistent with the
buffer.
V1->V2:
- Name the buffer-less request as zero-length request.
--
Viresh
Viresh Kumar (2):
virtio: i2c: No need to have separate read-write buffers
virtio: i2c: Allow zero-length transactions
virtio-i2c.tex | 90 +++++++++++++++++++++++++++++++-------------------
1 file changed, 56 insertions(+), 34 deletions(-)
--
2.31.1.272.g89b43f80a514
The I2C protocol allows zero-length requests with no data, like the
SMBus Quick command, where the command is inferred based on the
read/write flag itself.
In order to allow such a request, allocate another bit,
VIRTIO_I2C_FLAGS_M_RD(1), in the flags to pass the request type, as read
or write. This was earlier done using the read/write permission to the
buffer itself.
This still won't work well if multiple buffers are passed for the same
request, i.e. the write-read requests, as the VIRTIO_I2C_FLAGS_M_RD flag
can only be used with a single buffer.
Coming back to it, there is no need to send multiple buffers with a
single request. All we need, is a way to group several requests
together, which we can already do based on the
VIRTIO_I2C_FLAGS_FAIL_NEXT flag.
Remove support for multiple buffers within a single request.
Add a new feature flag for zero length requests and make it mandatory
for it to be implemented, so we don't need to drag the old
implementation around.
Fixes: https://github.com/oasis-tcs/virtio-spec/issues/112
Signed-off-by: Viresh Kumar <viresh.kumar(a)linaro.org>
---
V3->V4:
- Add a new mandatory feature flag.
V2->V3:
- Add conformance clauses that require that the flag is consistent with the
buffer.
V1->V2:
- Name the buffer-less request as zero-length request.
Hi Guys,
I did try to follow the discussion you guys had during V4, where we added
support for multiple buffers for the same request, which I think is unnecessary
now, after introduction of the VIRTIO_I2C_FLAGS_FAIL_NEXT flag.
https://lists.oasis-open.org/archives/virtio-comment/202011/msg00005.html
And so starting this discussion again, because we need to support stuff
like: i2cdetect -q <i2c-bus-number>, which issues a zero-length SMBus
Quick command.
---
virtio-i2c.tex | 76 ++++++++++++++++++++++++++++++--------------------
1 file changed, 45 insertions(+), 31 deletions(-)
diff --git a/virtio-i2c.tex b/virtio-i2c.tex
index 949d75f44158..0e73348963ce 100644
--- a/virtio-i2c.tex
+++ b/virtio-i2c.tex
@@ -17,7 +17,11 @@ \subsection{Virtqueues}\label{sec:Device Types / I2C Adapter Device / Virtqueues
\subsection{Feature bits}\label{sec:Device Types / I2C Adapter Device / Feature bits}
-None currently defined.
+\begin{description}
+\item[VIRTIO_I2C_F_ZERO_LENGTH_REQUEST (0)] The device supports zero-length I2C
+request and \field{VIRTIO_I2C_FLAGS_M_RD} flag. It is mandatory to implement
+this feature.
+\end{description}
\subsection{Device configuration layout}\label{sec:Device Types / I2C Adapter Device / Device configuration layout}
@@ -54,8 +58,7 @@ \subsubsection{Device Operation: Request Queue}\label{sec:Device Types / I2C Ada
\begin{lstlisting}
struct virtio_i2c_req {
struct virtio_i2c_out_hdr out_hdr;
- u8 write_buf[];
- u8 read_buf[];
+ u8 buf[];
struct virtio_i2c_in_hdr in_hdr;
};
\end{lstlisting}
@@ -84,16 +87,16 @@ \subsubsection{Device Operation: Request Queue}\label{sec:Device Types / I2C Ada
and sets it on the other requests. If this bit is set and a device fails
to process the current request, it needs to fail the next request instead
of attempting to execute it.
+
+\item[VIRTIO_I2C_FLAGS_M_RD(1)] is used to mark the request as READ or WRITE.
\end{description}
Other bits of \field{flags} are currently reserved as zero for future feature
extensibility.
-The \field{write_buf} of the request contains one segment of an I2C transaction
-being written to the device.
-
-The \field{read_buf} of the request contains one segment of an I2C transaction
-being read from the device.
+The \field{buf} of the request is optional and contains one segment of an I2C
+transaction being read from or written to the device, based on the value of the
+\field{VIRTIO_I2C_FLAGS_M_RD} bit in the \field{flags} field.
The final \field{status} byte of the request is written by the device: either
VIRTIO_I2C_MSG_OK for success or VIRTIO_I2C_MSG_ERR for error.
@@ -103,27 +106,27 @@ \subsubsection{Device Operation: Request Queue}\label{sec:Device Types / I2C Ada
#define VIRTIO_I2C_MSG_ERR 1
\end{lstlisting}
-If ``length of \field{read_buf}''=0 and ``length of \field{write_buf}''>0,
-the request is called write request.
+If \field{VIRTIO_I2C_FLAGS_M_RD} bit is set in the \field{flags}, then the
+request is called a read request.
-If ``length of \field{read_buf}''>0 and ``length of \field{write_buf}''=0,
-the request is called read request.
+If \field{VIRTIO_I2C_FLAGS_M_RD} bit is unset in the \field{flags}, then the
+request is called a write request.
-If ``length of \field{read_buf}''>0 and ``length of \field{write_buf}''>0,
-the request is called write-read request. It means an I2C write segment followed
-by a read segment. Usually, the write segment provides the number of an I2C
-controlled device register to be read.
+The \field{buf} is optional and will not be present for a zero-length request,
+like SMBus Quick.
-The case when ``length of \field{write_buf}''=0, and at the same time,
-``length of \field{read_buf}''=0 doesn't make any sense.
+The virtio I2C protocol supports write-read requests, i.e. an I2C write segment
+followed by a read segment (usually, the write segment provides the number of an
+I2C controlled device register to be read), by grouping a list of requests
+together using the \field{VIRTIO_I2C_FLAGS_FAIL_NEXT} flag.
\subsubsection{Device Operation: Operation Status}\label{sec:Device Types / I2C Adapter Device / Device Operation: Operation Status}
-\field{addr}, \field{flags}, ``length of \field{write_buf}'' and ``length of \field{read_buf}''
-are determined by the driver, while \field{status} is determined by the processing
-of the device. A driver puts the data written to the device into \field{write_buf}, while
-a device puts the data of the corresponding length into \field{read_buf} according to the
-request of the driver.
+\field{addr}, \field{flags}, and ``length of \field{buf}'' are determined by the
+driver, while \field{status} is determined by the processing of the device. A
+driver, for a write request, puts the data to be written to the device into the
+\field{buf}, while a device, for a read request, puts the data read from device
+into the \field{buf} according to the request from the driver.
A driver may send one request or multiple requests to the device at a time.
The requests in the virtqueue are both queued and processed in order.
@@ -137,15 +140,22 @@ \subsubsection{Device Operation: Operation Status}\label{sec:Device Types / I2C
\drivernormative{\subsubsection}{Device Operation}{Device Types / I2C Adapter Device / Device Operation}
+A driver SHOULD implement the VIRTIO_I2C_F_ZERO_LENGTH_REQUEST feature.
+
A driver MUST set \field{addr} and \field{flags} before sending the request.
A driver MUST set the reserved bits of \field{flags} to be zero.
-The driver MUST NOT send a request with ``length of \field{write_buf}''=0 and
-``length of \field{read_buf}''=0 at the same time.
+A driver MUST NOT send the \field{buf}, for a zero-length request.
+
+A driver MUST NOT use \field{buf}, for a read request, if the final
+\field{status} returned from the device is VIRTIO_I2C_MSG_ERR.
+
+A driver MUST set the \field{VIRTIO_I2C_FLAGS_M_RD} flag for a read operation,
+where the buffer is write-only for the device.
-A driver MUST NOT use \field{read_buf} if the final \field{status} returned
-from the device is VIRTIO_I2C_MSG_ERR.
+A driver MUST NOT set the \field{VIRTIO_I2C_FLAGS_M_RD} flag for a write
+operation, where the buffer is read-only for the device.
A driver MUST queue the requests in order if multiple requests are going to
be sent at a time.
@@ -157,14 +167,18 @@ \subsubsection{Device Operation: Operation Status}\label{sec:Device Types / I2C
\devicenormative{\subsubsection}{Device Operation}{Device Types / I2C Adapter Device / Device Operation}
+A device SHOULD implement the VIRTIO_I2C_F_ZERO_LENGTH_REQUEST feature.
+
A device SHOULD keep consistent behaviors with the hardware as described in
\hyperref[intro:I2C]{I2C}.
-A device MUST NOT change the value of \field{addr}, reserved bits of \field{flags}
-and \field{write_buf}.
+A device MUST NOT change the value of \field{addr}, and reserved bits of
+\field{flags}.
+
+A device MUST not change the value of the \field{buf} for a write request.
-A device MUST place one I2C segment of the corresponding length into \field{read_buf}
-according the driver's request.
+A device MUST place one I2C segment of the ``length of \field{buf}'', for the
+read request, into the \field{buf} according the driver's request.
A device MUST guarantee the requests in the virtqueue being processed in order
if multiple requests are received at a time.
--
2.31.1.272.g89b43f80a514