From 20a5ad3ec1c4e5a29bc21552f6d45b8debb203d6 Mon Sep 17 00:00:00 2001 From: Josh Boyer Date: Sat, 9 Nov 2013 09:38:53 -0500 Subject: [PATCH] Add patch from Daniel Stone to avoid high order allocations in evdev --- ...k-to-vmalloc-for-client-event-buffer.patch | 64 +++++++++++++++++++ kernel.spec | 5 ++ 2 files changed, 69 insertions(+) create mode 100644 Input-evdev-fall-back-to-vmalloc-for-client-event-buffer.patch diff --git a/Input-evdev-fall-back-to-vmalloc-for-client-event-buffer.patch b/Input-evdev-fall-back-to-vmalloc-for-client-event-buffer.patch new file mode 100644 index 000000000..da1b92ed8 --- /dev/null +++ b/Input-evdev-fall-back-to-vmalloc-for-client-event-buffer.patch @@ -0,0 +1,64 @@ +From 92eb77d0ffbaa71b501a0a8dabf09a351bf4267f Mon Sep 17 00:00:00 2001 +From: Daniel Stone +Date: Thu, 31 Oct 2013 07:25:34 +0000 +Subject: Input: evdev - fall back to vmalloc for client event buffer + +evdev always tries to allocate the event buffer for clients using +kzalloc rather than vmalloc, presumably to avoid mapping overhead where +possible. However, drivers like bcm5974, which claims support for +reporting 16 fingers simultaneously, can have an extraordinarily large +buffer. The resultant contiguous order-4 allocation attempt fails due +to fragmentation, and the device is thus unusable until reboot. + +Try kzalloc if we can to avoid the mapping overhead, but if that fails, +fall back to vzalloc. + +Signed-off-by: Daniel Stone +Signed-off-by: Dmitry Torokhov +--- +diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c +index b6ded17..a06e125 100644 +--- a/drivers/input/evdev.c ++++ b/drivers/input/evdev.c +@@ -18,6 +18,8 @@ + #include + #include + #include ++#include ++#include + #include + #include + #include +@@ -369,7 +371,11 @@ static int evdev_release(struct inode *inode, struct file *file) + mutex_unlock(&evdev->mutex); + + evdev_detach_client(evdev, client); +- kfree(client); ++ ++ if (is_vmalloc_addr(client)) ++ vfree(client); ++ else ++ kfree(client); + + evdev_close_device(evdev); + +@@ -389,12 +395,14 @@ static int evdev_open(struct inode *inode, struct file *file) + { + struct evdev *evdev = container_of(inode->i_cdev, struct evdev, cdev); + unsigned int bufsize = evdev_compute_buffer_size(evdev->handle.dev); ++ unsigned int size = sizeof(struct evdev_client) + ++ bufsize * sizeof(struct input_event); + struct evdev_client *client; + int error; + +- client = kzalloc(sizeof(struct evdev_client) + +- bufsize * sizeof(struct input_event), +- GFP_KERNEL); ++ client = kzalloc(size, GFP_KERNEL | __GFP_NOWARN); ++ if (!client) ++ client = vzalloc(size); + if (!client) + return -ENOMEM; + +-- +cgit v0.9.2 diff --git a/kernel.spec b/kernel.spec index 1ae399616..9842b33e3 100644 --- a/kernel.spec +++ b/kernel.spec @@ -740,6 +740,8 @@ Patch25139: net-flow_dissector-fail-on-evil-iph-ihl.patch Patch25140: drm-qxl-backport-fixes-for-Fedora.patch +Patch25141: Input-evdev-fall-back-to-vmalloc-for-client-event-buffer.patch + # END OF PATCH DEFINITIONS %endif @@ -1448,6 +1450,8 @@ ApplyPatch net-flow_dissector-fail-on-evil-iph-ihl.patch ApplyPatch drm-qxl-backport-fixes-for-Fedora.patch +ApplyPatch Input-evdev-fall-back-to-vmalloc-for-client-event-buffer.patch + # END OF PATCH APPLICATIONS %endif @@ -2252,6 +2256,7 @@ fi # || || %changelog * Sat Nov 09 2013 Josh Boyer +- Add patch from Daniel Stone to avoid high order allocations in evdev - Add qxl backport fixes from Dave Airlie * Tue Nov 05 2013 Kyle McMartin