Date: Sun, 30 Aug 2009 08:44:28 GMT From: Alexander Motin <mav@FreeBSD.org> To: Perforce Change Reviews <perforce@freebsd.org> Subject: PERFORCE change 167966 for review Message-ID: <200908300844.n7U8iSYP053736@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=167966 Change 167966 by mav@mav_mavbook on 2009/08/30 08:43:32 Remove msleep() timeout from g_io_schedule_up/down(). It works fine without it, saving few percents of CPU on high request rates without need to rearm callout twice per request, Affected files ... .. //depot/projects/scottl-camlock/src/sys/geom/geom_io.c#17 edit Differences ... ==== //depot/projects/scottl-camlock/src/sys/geom/geom_io.c#17 (text+ko) ==== @@ -567,7 +567,7 @@ if (bp == NULL) { CTR0(KTR_GEOM, "g_down going to sleep"); msleep(&g_wait_down, &g_bio_run_down.bio_queue_lock, - PRIBIO | PDROP, "-", hz/10); + PRIBIO | PDROP, "-", 0); continue; } CTR0(KTR_GEOM, "g_down has work to do"); @@ -672,7 +672,7 @@ } CTR0(KTR_GEOM, "g_up going to sleep"); msleep(&g_wait_up, &g_bio_run_up.bio_queue_lock, - PRIBIO | PDROP, "-", hz/10); + PRIBIO | PDROP, "-", 0); } }
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200908300844.n7U8iSYP053736>