From 99f4e6a8b21c182fba9634b15f2afdd508b3b867 Mon Sep 17 00:00:00 2001 From: Josh Boyer Date: Nov 09 2013 14:40:25 +0000 Subject: Add patch from Daniel Stone to avoid high order allocations in evdev --- 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 0000000..da1b92e --- /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 0dcb195..eba97f3 100644 --- a/kernel.spec +++ b/kernel.spec @@ -804,6 +804,8 @@ Patch25142: iwlwifi-dvm-dont-override-mac80211-queue-setting.patch Patch25143: drm-qxl-backport-fixes-for-Fedora.patch +Patch25144: Input-evdev-fall-back-to-vmalloc-for-client-event-buffer.patch + # END OF PATCH DEFINITIONS %endif @@ -1545,6 +1547,8 @@ ApplyPatch iwlwifi-dvm-dont-override-mac80211-queue-setting.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 @@ -2387,6 +2391,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 * Mon Nov 04 2013 Josh Boyer - 3.11.7-100