On 01/30/16 19:31, Slawa Olhovchenkov wrote:
On Sat, Jan 30, 2016 at 09:25:21PM +0300, Slawa Olhovchenkov wrote:

On Sat, Jan 30, 2016 at 09:42:13PM +0330, mokhi wrote:

i currently only wanna do patch on kbd.c (because i'm sure there is a
thread-unsafety)
and i don't want to add anything to spltty() nor splx(), i just wanna
add things under where they've been used.
isn't problem with using mutex/spin/lock/unlock etc there?

yes.
currenly spltty act as mutex_lock(&spltty_mutex), like global lock for
all used files. this is also inter-files lock, not only lock inside
only kbd.c.
you need patch all files. all using spltty need to remove
simultaneously.


oh, sorry.
i am see now all splXXX is nop.
may be incorretly -- for example, in digi.c i am don't see any lockind
instead spltty().

Hi,

Code that uses splxxx() needs to have the Giant lock locked when running. Refer to the various MPSAFE flags.

--HPS

_______________________________________________
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Reply via email to