Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 21 Jan 2003 18:21:55 -0800
From:      Alfred Perlstein <alfred@FreeBSD.org>
To:        Peter Wemm <peter@wemm.org>
Cc:        Jeff Roberson <jroberson@chesapeake.net>, Bosko Milekic <bmilekic@unixdaemons.com>, Nate Lawson <nate@root.org>, Bruce Evans <bde@zeta.org.au>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/alpha/alpha busdma_machdep.c src/sys/alpha/osf1 imgact_osf1.c osf1_misc.c src/sys/cam cam_periph.c cam_sim.c cam_xpt.c src/sys/cam/scsi scsi_cd.c scsi_ch.c scsi
Message-ID:  <20030122022155.GO42333@elvis.mu.org>
In-Reply-To: <20030122020826.87F7F2A7EA@canning.wemm.org>
References:  <20030121205616.I46974-100000@mail.chesapeake.net> <20030122020826.87F7F2A7EA@canning.wemm.org>

next in thread | previous in thread | raw e-mail | index | archive | help
* Peter Wemm <peter@wemm.org> [030121 18:08] wrote:
> Jeff Roberson wrote:
> > 
> > Ok, I looked at the diff/commit a little closer.  What I suggest we do is
> > this:
> > 
> > Leave M_WAITOK defined.  This way we keep the ABI and API the same for
> > 5.0.  In 6.0 go ahead and remove it.  New code should not use this 'flag'.
> 
> Slight variation.  Can we make M_WAITOK and the other not-a-flag flags
> ifdef'ed under KLD_MODULE or something?  That way we can reduce some of the
> avoidable pain with 3rd party external modules, and still get to be sure
> that our own house (src/sys/*) is kept clean.

That's fine with me, I wish there was a way to do

#ifused M_WAITOK
#warning ...

though. :)  I agree with your follow-up that people should be able to
suck it up with thier deltas though and deal with it without much of a
problem.

-- 
-Alfred Perlstein [alfred@freebsd.org]

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




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