[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sun, 03 Mar 2013 17:47:39 -0500 (EST)
From: David Miller <davem@...emloft.net>
To: sasha.levin@...cle.com
Cc: samuel@...tiz.org, gregkh@...uxfoundation.org, jslaby@...e.cz,
peter@...leysoftware.com, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ircomm: release tty before sleeping potentially
indefintely
From: Sasha Levin <sasha.levin@...cle.com>
Date: Sun, 3 Mar 2013 17:35:53 -0500
> ircomm_tty_block_til_ready would hold tty lock while blocking. Since the sleep
> might take a long time we can prevent other processes from accessing the tty,
> causing hung tasks and a dead tty.
>
> Diagnosed-by: Peter Hurley <peter@...leysoftware.com>
> Signed-off-by: Sasha Levin <sasha.levin@...cle.com>
But then you invalidate all of the tty state tests made under
the lock at the beginning of this function, before enterring
the loop. If you drop the lock, those pieces of state could
change.
I'm not applying this.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists