Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 6 Mar 2002 21:33:44 -0800 (PST)
From:      Matthew Dillon <dillon@apollo.backplane.com>
To:        "Justin T. Gibbs" <gibbs@scsiguy.com>
Cc:        John Baldwin <jhb@FreeBSD.ORG>, Bruce Evans <bde@zeta.org.au>, Terry Lambert <tlambert2@mindspring.com>, Alfred Perlstein <alfred@FreeBSD.ORG>, Bosko Milekic <bmilekic@unixdaemons.com>, Seigo Tanimura <tanimura@r.dl.itc.u-tokyo.ac.jp>, FreeBSD current users <current@FreeBSD.ORG>, Julian Elischer <julian@elischer.org>
Subject:   Re: Patch for critical_enter()/critical_exit() & interrupt assem 
Message-ID:  <200203070533.g275Xii64069@apollo.backplane.com>
References:   <200203070429.g274TlI51515@aslan.scsiguy.com>

next in thread | previous in thread | raw e-mail | index | archive | help
:My 2 cents?  Work with John to get the APIs that affect this particular
:code stable.  That means discussion and perhaps that discussion will
:take some time (if this blow-up hadn't occurred the discussion
:would already be over now ;-).  Once the APIs are in place, commit your
:code in a format that makes it completely optional, just like your Giant
:lock optimizations.  This means that those interested in validating
:and determining the impact of your code can easily do so by flipping
:a switch.  Those who would rather debug their own subsystem problems
:in a slower, but simpler, environment can do so too.
:
:--
:Justin

    The API is intended for the stage-2 commit.  The stage-1 commit
    is intended to do all the 'hard stuff' in as straightforward
    a manner as possible.  The sysctl / option you talk about here
    existed in my patch set 2 and a half weeks ago.

    I really wish you people would at least read the patch and commit
    message before you comment on it.

						-Matt


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200203070533.g275Xii64069>