* [PATCH v2] udev: add rules for qemu guests
@ 2011-01-25 14:51 Gerd Hoffmann
2011-01-25 15:14 ` Kay Sievers
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Gerd Hoffmann @ 2011-01-25 14:51 UTC (permalink / raw)
To: linux-hotplug
These patches enable usb autosuspend for the qemu emulated HID devices.
This reduces the cpu load for idle guests with a hid device attached
because the linux kernel will suspend the usb bus then and qemu can stop
running a 1000 Hz to emulate the (active) UHCI controller.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
Makefile.am | 5 +++++
| 13 +++++++++++++
rules/misc/30-kernel-compat.rules | 5 +++++
3 files changed, 23 insertions(+), 0 deletions(-)
create mode 100644 extras/qemu/42-qemu-usb.rules
diff --git a/Makefile.am b/Makefile.am
index 2e3fe39..b2300ce 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -317,6 +317,11 @@ extras_v4l_id_v4l_id_LDADD = libudev/libudev-private.la
libexec_PROGRAMS += extras/v4l_id/v4l_id
dist_udevrules_DATA += extras/v4l_id/60-persistent-v4l.rules
+# ------------------------------------------------------------------------------
+# qemu -- qemu/kvm guest tweaks
+# ------------------------------------------------------------------------------
+dist_udevrules_DATA += extras/qemu/99-qemu-usb.rules
+
if ENABLE_EXTRAS
# ------------------------------------------------------------------------------
# conditional extras (need glib, libusb, libacl, ...)
--git a/extras/qemu/42-qemu-usb.rules b/extras/qemu/42-qemu-usb.rules
new file mode 100644
index 0000000..4fd53c7
--- /dev/null
+++ b/extras/qemu/42-qemu-usb.rules
@@ -0,0 +1,13 @@
+#
+# Enable autosuspend for qemu emulated usb hid devices.
+#
+# Note that there are buggy qemu versions which advertise remote
+# wakeup support but don't actually implement it correctly. This
+# is the reason why we need a match for the serial number here.
+# The serial number "42" is used to tag the implementations where
+# remote wakeup is working.
+#
+
+ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Mouse", ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
+ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Tablet", ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
+CTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Keyboard", ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
diff --git a/rules/misc/30-kernel-compat.rules b/rules/misc/30-kernel-compat.rules
index 2c545dc..2a6a97c 100644
--- a/rules/misc/30-kernel-compat.rules
+++ b/rules/misc/30-kernel-compat.rules
@@ -97,4 +97,9 @@ SUBSYSTEM="mmc", RUN+="/sbin/modprobe -b mmc_block"
SUBSYSTEM="pnp", DRIVER!="?*", \
RUN{ignore_error}+="/bin/sh -c '/sbin/modprobe -a $$(while read id; do echo pnp:d$$id; done < /sys$devpath/id)'"
+# see extras/qemu/42-qemu-usb.rules, version for 2.6.32 + older.
+ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Mouse", ATTR{serial}="42", TEST="power/level", ATTR{power/level}="auto"
+ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Tablet", ATTR{serial}="42", TEST="power/level", ATTR{power/level}="auto"
+ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Keyboard", ATTR{serial}="42", TEST="power/level", ATTR{power/level}="auto"
+
LABEL="kernel_compat_end"
--
1.7.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] udev: add rules for qemu guests
2011-01-25 14:51 [PATCH v2] udev: add rules for qemu guests Gerd Hoffmann
@ 2011-01-25 15:14 ` Kay Sievers
2011-01-25 18:43 ` Ozan Çağlayan
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Kay Sievers @ 2011-01-25 15:14 UTC (permalink / raw)
To: linux-hotplug
On Tue, Jan 25, 2011 at 15:51, Gerd Hoffmann <kraxel@redhat.com> wrote:
> These patches enable usb autosuspend for the qemu emulated HID devices.
> This reduces the cpu load for idle guests with a hid device attached
> because the linux kernel will suspend the usb bus then and qemu can stop
> running a 1000 Hz to emulate the (active) UHCI controller.
Applied.
Thanks,
Kay
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] udev: add rules for qemu guests
2011-01-25 14:51 [PATCH v2] udev: add rules for qemu guests Gerd Hoffmann
2011-01-25 15:14 ` Kay Sievers
@ 2011-01-25 18:43 ` Ozan Çağlayan
2011-01-25 20:47 ` Kay Sievers
2011-01-26 4:59 ` Jim Paris
3 siblings, 0 replies; 5+ messages in thread
From: Ozan Çağlayan @ 2011-01-25 18:43 UTC (permalink / raw)
To: linux-hotplug
On 25.01.2011 16:51, Gerd Hoffmann wrote:
> +ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Mouse", ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
> +ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Tablet", ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
> +CTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Keyboard", ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
There's a typo in here.
--
Pardus Linux
http://www.pardus.org.tr/eng
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] udev: add rules for qemu guests
2011-01-25 14:51 [PATCH v2] udev: add rules for qemu guests Gerd Hoffmann
2011-01-25 15:14 ` Kay Sievers
2011-01-25 18:43 ` Ozan Çağlayan
@ 2011-01-25 20:47 ` Kay Sievers
2011-01-26 4:59 ` Jim Paris
3 siblings, 0 replies; 5+ messages in thread
From: Kay Sievers @ 2011-01-25 20:47 UTC (permalink / raw)
To: linux-hotplug
On Tue, Jan 25, 2011 at 19:43, Ozan Çağlayan <ozan@pardus.org.tr> wrote:
> On 25.01.2011 16:51, Gerd Hoffmann wrote:
>
>> +ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Mouse",
>> ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
>> +ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Tablet",
>> ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
>> +CTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Keyboard",
>> ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
>
> There's a typo in here.
Fixed.
Thanks,
Kay
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] udev: add rules for qemu guests
2011-01-25 14:51 [PATCH v2] udev: add rules for qemu guests Gerd Hoffmann
` (2 preceding siblings ...)
2011-01-25 20:47 ` Kay Sievers
@ 2011-01-26 4:59 ` Jim Paris
3 siblings, 0 replies; 5+ messages in thread
From: Jim Paris @ 2011-01-26 4:59 UTC (permalink / raw)
To: linux-hotplug
Gerd Hoffmann wrote:
> These patches enable usb autosuspend for the qemu emulated HID devices.
> This reduces the cpu load for idle guests with a hid device attached
> because the linux kernel will suspend the usb bus then and qemu can stop
> running a 1000 Hz to emulate the (active) UHCI controller.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
> Makefile.am | 5 +++++
> extras/qemu/42-qemu-usb.rules | 13 +++++++++++++
> rules/misc/30-kernel-compat.rules | 5 +++++
> 3 files changed, 23 insertions(+), 0 deletions(-)
> create mode 100644 extras/qemu/42-qemu-usb.rules
>
> diff --git a/Makefile.am b/Makefile.am
> index 2e3fe39..b2300ce 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -317,6 +317,11 @@ extras_v4l_id_v4l_id_LDADD = libudev/libudev-private.la
> libexec_PROGRAMS += extras/v4l_id/v4l_id
> dist_udevrules_DATA += extras/v4l_id/60-persistent-v4l.rules
>
> +# ------------------------------------------------------------------------------
> +# qemu -- qemu/kvm guest tweaks
> +# ------------------------------------------------------------------------------
> +dist_udevrules_DATA += extras/qemu/99-qemu-usb.rules
That should be 42-qemu-usb.rules
-jim
> +
> if ENABLE_EXTRAS
> # ------------------------------------------------------------------------------
> # conditional extras (need glib, libusb, libacl, ...)
> diff --git a/extras/qemu/42-qemu-usb.rules b/extras/qemu/42-qemu-usb.rules
> new file mode 100644
> index 0000000..4fd53c7
> --- /dev/null
> +++ b/extras/qemu/42-qemu-usb.rules
> @@ -0,0 +1,13 @@
> +#
> +# Enable autosuspend for qemu emulated usb hid devices.
> +#
> +# Note that there are buggy qemu versions which advertise remote
> +# wakeup support but don't actually implement it correctly. This
> +# is the reason why we need a match for the serial number here.
> +# The serial number "42" is used to tag the implementations where
> +# remote wakeup is working.
> +#
> +
> +ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Mouse", ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
> +ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Tablet", ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
> +CTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Keyboard", ATTR{serial}="42", TEST="power/control", ATTR{power/control}="auto"
> diff --git a/rules/misc/30-kernel-compat.rules b/rules/misc/30-kernel-compat.rules
> index 2c545dc..2a6a97c 100644
> --- a/rules/misc/30-kernel-compat.rules
> +++ b/rules/misc/30-kernel-compat.rules
> @@ -97,4 +97,9 @@ SUBSYSTEM="mmc", RUN+="/sbin/modprobe -b mmc_block"
> SUBSYSTEM="pnp", DRIVER!="?*", \
> RUN{ignore_error}+="/bin/sh -c '/sbin/modprobe -a $$(while read id; do echo pnp:d$$id; done < /sys$devpath/id)'"
>
> +# see extras/qemu/42-qemu-usb.rules, version for 2.6.32 + older.
> +ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Mouse", ATTR{serial}="42", TEST="power/level", ATTR{power/level}="auto"
> +ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Tablet", ATTR{serial}="42", TEST="power/level", ATTR{power/level}="auto"
> +ACTION="add", SUBSYSTEM="usb", ATTR{product}="QEMU USB Keyboard", ATTR{serial}="42", TEST="power/level", ATTR{power/level}="auto"
> +
> LABEL="kernel_compat_end"
> --
> 1.7.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-01-26 4:59 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-25 14:51 [PATCH v2] udev: add rules for qemu guests Gerd Hoffmann
2011-01-25 15:14 ` Kay Sievers
2011-01-25 18:43 ` Ozan Çağlayan
2011-01-25 20:47 ` Kay Sievers
2011-01-26 4:59 ` Jim Paris
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.