On Fri, Mar 31, 2017 at 02:20:27PM +0200, Hans Verkuil wrote:
> +struct cec_notifier *cec_notifier_get(struct device *dev)
> +{
> +     struct cec_notifier *n;
> +
> +     mutex_lock(&cec_notifiers_lock);
> +     list_for_each_entry(n, &cec_notifiers, head) {
> +             if (n->dev == dev) {
> +                     mutex_unlock(&cec_notifiers_lock);
> +                     kref_get(&n->kref);

Isn't this racy?  What stops one thread trying to get the notifier
while another thread puts the notifier?

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

Reply via email to