Thread (6 messages) 6 messages, 2 authors, 2007-07-03

Re: [PATCH] genericserial: Remove bogus optimisation check and dead code paths

From: Alan Cox <hidden>
Date: 2007-07-03 22:25:29
Also in: lkml

On Tue, 03 Jul 2007 23:36:22 +0200
Morten Helgesen [off-list ref] wrote:

 /*
 > Problems to take into account are:
 >       -1- Interrupts that empty part of the buffer.
->       -2- page faults on the access to userspace. 
+>       -2- Page faults on access to userspace. 
#2 no longer applies (don't before the caller)
 >       -3- Other processes that are also trying to do a "write".   
#3 no longer applies (locked by tty layer)
-	/* get exclusive "write" access to this port (problem 3) */
+	/* Get exclusive "write" access to this port (problem 3) */
 	/* This is not a spinlock because we can have a disk access (page 
-		 fault) in copy_from_user */
No copy_from_user any more. Actually that locking code wants review and
maybe removal so is another project altogether


Definitely an improvement

Acked-by: Alan Cox <redacted>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help