Date: Mon, 15 Feb 2010 23:48:43 +0100 From: Attilio Rao <attilio@freebsd.org> To: Andriy Gapon <avg@freebsd.org> Cc: svn-src-head@freebsd.org, Alexey Dokuchaev <danfe@freebsd.org>, src-committers@freebsd.org, svn-src-all@freebsd.org Subject: Re: svn commit: r203758 - in head/sys: amd64/conf arm/conf i386/conf ia64/conf mips/conf pc98/conf powerpc/conf sparc64/conf sun4v/conf Message-ID: <3bbf2fe11002151448t6ca9a519s65fda66af3927086@mail.gmail.com> In-Reply-To: <4B79CE84.3060901@freebsd.org> References: <201002101630.o1AGU4kn043737@svn.freebsd.org> <20100210203519.GA44831@FreeBSD.org> <3bbf2fe11002151411u552a4749j847b3e10645293e7@mail.gmail.com> <4B79CE84.3060901@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
2010/2/15 Andriy Gapon <avg@freebsd.org>: > on 16/02/2010 00:11 Attilio Rao said the following: >> 2010/2/10 Alexey Dokuchaev <danfe@freebsd.org>: >>> On Wed, Feb 10, 2010 at 04:30:04PM +0000, Attilio Rao wrote: >>>> Author: attilio >>>> Date: Wed Feb 10 16:30:04 2010 >>>> New Revision: 203758 >>>> URL: http://svn.freebsd.org/changeset/base/203758 >>>> >>>> Log: >>>> =C2=A0 Add the options DEADLKRES (introducing the deadlock resolver th= read) in >>>> =C2=A0 the 'debugging' section of any HEAD kernel and enable for the m= ainstream >>>> =C2=A0 ones, excluding the embedded architectures. >>>> =C2=A0 It may, of course, enabled on a case-by-case basis. >>>> >>>> =C2=A0 Sponsored by: =C2=A0 =C2=A0 =C2=A0 Sandvine Incorporated >>>> =C2=A0 Requested by: =C2=A0 =C2=A0 =C2=A0 emaste >>>> =C2=A0 Discussed with: =C2=A0 =C2=A0 kib >>>> >>>> @@ -72,6 +72,7 @@ options =C2=A0 =C2=A0 INCLUDE_CONFIG_FILE =C2=A0 =C2= =A0 # Inclu >>>> =C2=A0options =C2=A0 =C2=A0 =C2=A0KDB =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 # Enable kernel debugger support. >>>> =C2=A0options =C2=A0 =C2=A0 =C2=A0DDB =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 # Support DDB. >>>> =C2=A0options =C2=A0 =C2=A0 =C2=A0GDB =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 # Support remote GDB. >>>> +options =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0DEADLKRES =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 # Enable the deadlock resolve= r >>>> =C2=A0options =C2=A0 =C2=A0 =C2=A0INVARIANTS =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0# Enable calls of extra sanity checking >>> Garbled indentation suggests you've put <tab> after "options" instead o= f >>> space. =C2=A0If you decide to fix it, you can take care of the similar = error >>> with FLOWTABLE option at least in amd64/conf/GENERIC. >> >> I'm sorry but it seems that all the other (at least for debugging) are >> as 'options<tab>MACRO' which is what I did here, or I made something >> wrong? > > At least in the i386/amd64 kernel config files typically we have: > options<space><tab>OPTION > And it is the case for the quoted above amd64 context. Oh, I see now, I did <tab><tab> opposed to <space><tab>. Thanks, Attilio --=20 Peace can only be achieved by understanding - A. Einstein
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3bbf2fe11002151448t6ca9a519s65fda66af3927086>