| 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
| ||
|
Date: Wed, 31 Oct 2012 14:05:32 -0700 From: Tejun Heo <tj@...nel.org> To: Dmitry Torokhov <dmitry.torokhov@...il.com> Cc: linux-input@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH] Input: introduce managed input devices (add devres support) Hello, Dmitry. On Mon, Oct 22, 2012 at 10:35:14PM -0700, Dmitry Torokhov wrote: > There is a demand from driver's writers to use managed devices framework > for their drivers. Unfortunately up to this moment input devices did not > provide support for managed devices and that lead to mixing two styles > of resource management which usually introduced more bugs, such as > manually unregistering input device but relying in devres to free > interrupt handler which (unless device is properly shut off) can cause > ISR to reference already freed memory. > > This change introduces devm_input_allocate_device() that will allocate > managed instance of input device so that driver writers who prefer > using devm_* framework do not have to mix 2 styles. It generally looks good to me although it's a bit unusual to use devres on device. Given the way input_dev is used, it probably makes sense, I guess. One thing tho. If possible and there aren't too many, wouldn't it be better to convert all to use devres. Having two different lifetime fules tends to lead to gotchas. Thanks! -- tejun -- 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