Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 3 Mar 2016 13:29:08 +0200
From:      Konstantin Belousov <kostikbel@gmail.com>
To:        Hans Petter Selasky <hps@selasky.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r296320 - in head/sys: kern sys
Message-ID:  <20160303112908.GP67250@kib.kiev.ua>
In-Reply-To: <56D81A70.2070508@selasky.org>
References:  <201603021846.u22IkHWM010861@repo.freebsd.org> <56D81918.6020403@selasky.org> <56D81A70.2070508@selasky.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Mar 03, 2016 at 12:05:20PM +0100, Hans Petter Selasky wrote:
> On 03/03/16 11:59, Hans Petter Selasky wrote:
> > +/* Flags for callout_stop_safe() */
> > +#define    CS_DRAIN        0x0001 /* callout_drain(), wait allowed */
> > +#define    CS_MIGRBLOCK        0x0002 /* Block migration, return value
> > +                      indicates that the callout was
> > +                          executing */
> > +
> >    #ifdef _KERNEL
> 
> Should these defines be under #ifdef _KERNEL ?

Why do they need the braces ?  Other bits in the callout.h header are
not restricted for kernel only, and I do not see a difference between
CALLOUT_* and CS_*.



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