Date: Thu, 22 Jul 2010 14:15:22 -0700 From: Jerry Toung <jrytoung@gmail.com> To: freebsd-hackers@freebsd.org Subject: Giant free GEOM/CAM XPT Message-ID: <AANLkTinQb0wX38ahqHi9fX7APZSQfOESAJRpQlXPEv5U@mail.gmail.com>
next in thread | raw e-mail | index | archive | help
Hello List, while going through the xpt code (8.0 RELEASE), it seems to me that some gains can be had in src/sys/geom/geom_disk.c where dp->d_strategy(bp2) is surrounded by Giant lock. Especially in the case where one has 2+ controllers on the system with /dev/daXX attached to them during heavy I/O. I am currently trying to get rid of giant there, but it branches in sys/cam and sys/dev/twa. Definitely not a trivial exercise. The dependency on Giant seems to come from the XPT code. would be neat if I could just use the SIM lock, which is per controller. Question: do you think it's worth the effort? right now I always get a crash (race condition most likely) in xpt_run_dev_allocq->camq_remove Jerry
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTinQb0wX38ahqHi9fX7APZSQfOESAJRpQlXPEv5U>