Date: Sun, 24 Oct 2010 18:47:00 +0300 From: Jaakko Heinonen <jh@FreeBSD.org> To: Pawel Jakub Dawidek <pjd@FreeBSD.org> Cc: Ivan Voras <ivoras@freebsd.org>, freebsd-geom@freebsd.org Subject: Re: Hyperactive g_event thread Message-ID: <20101024154700.GA1386@a91-153-123-205.elisa-laajakaista.fi> In-Reply-To: <20101022210317.GB1742@garage.freebsd.pl> References: <4C4ED619.7050305@FreeBSD.org> <27237.1280241532@critter.freebsd.dk> <AANLkTi=uRPV2T0=t_1s=Jc4PmBtai=__HqhCtYpiDdTQ@mail.gmail.com> <AANLkTikvV4oymBBA%2B_0zbzd_edS8dRfqqJRODE0989%2Bn@mail.gmail.com> <4C4F171C.9010106@FreeBSD.org> <20101021174301.GA1381@a91-153-123-205.elisa-laajakaista.fi> <i9rn51$ec0$1@dough.gmane.org> <20101022184645.GA1381@a91-153-123-205.elisa-laajakaista.fi> <20101022210317.GB1742@garage.freebsd.pl>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2010-10-22, Pawel Jakub Dawidek wrote: > You won't lose wakeup even if you unlock first and then call wakeup(), > but unlocking first might save you a context switch. Thanks, I see it now. I reverted the wakeup() and unlock ordering in g_orphan_provider() and g_post_event_x(). Also g_do_wither() unlocks before calling wakeup() now. An updated patch: http://people.freebsd.org/~jh/patches/geom-eventproc-sleep.2.diff -- Jaakko
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20101024154700.GA1386>