| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <55DBED78.6060709@redhat.com> Date: Mon, 24 Aug 2015 21:22:16 -0700 From: Laura Abbott <labbott@...hat.com> To: Dmitry Torokhov <dmitry.torokhov@...il.com>, Laura Abbott <labbott@...oraproject.org> Cc: linux-input@...r.kernel.org, linux-kernel@...r.kernel.org, linux-usb@...r.kernel.org Subject: Re: [PATCHv2] Input: xpad - Fix double URB submission races On 08/21/2015 09:50 AM, Dmitry Torokhov wrote: > Hi Laura, > > On Mon, Aug 10, 2015 at 05:26:12PM -0700, Laura Abbott wrote: >> v2: Created a proper queue for events instead of just dropping them > > How long does it take for the queue to exhaust your memory if you keep > bombarding the driver with requests? > My script which changes the LEDs as fast as possible ran for 7+ hours on my machine with 16GB of RAM without exhausting all of it. This is also a very extreme case as almost any kind of delay between sending commands will drain the queue. > I do not think you need a queue. I believe the nature of LEDs and rumble > force feedback effect is such that you can discard all requests but the > latest that arrived between the moment you submitted a request to the > device and the moment you are ready submit a new one. So your suggestion is to only keep a single item in the queue? > > Thanks. > Thanks, Laura -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists