Re: CVS commit: src/tests/kernel

2013-02-19 Thread Joerg Sonnenberger
On Mon, Feb 18, 2013 at 07:24:39PM -0800, Paul Goyette wrote: > On Tue, 19 Feb 2013, Joerg Sonnenberger wrote: > > >>Wait a little bit after fork()ing the lockers to give them a chance to > >>get started before trying to ptrace(ATTACH). > > > >What about using a pipe to make this an explicit barri

Re: CVS commit: src/tests/kernel

2013-02-19 Thread Paul Goyette
Most of the printf's were already there before this round of updates, and there's enough differentiation in them to figure out which process is responsible. In any case, the printf's aren't critical to the test (until you need to debug it!). So for now, I'd rather just leave them all alone.

Re: CVS commit: src/tests/kernel

2013-02-19 Thread Joerg Sonnenberger
On Tue, Feb 19, 2013 at 11:40:22AM -0800, Paul Goyette wrote: > Most of the printf's were already there before this round of > updates, and there's enough differentiation in them to figure out > which process is responsible. In any case, the printf's aren't > critical to the test (until you need t

Re: CVS commit: src/tests/kernel

2013-02-19 Thread Julio Merino
On Tue, Feb 19, 2013 at 2:43 PM, Joerg Sonnenberger wrote: > On Tue, Feb 19, 2013 at 11:40:22AM -0800, Paul Goyette wrote: >> Most of the printf's were already there before this round of >> updates, and there's enough differentiation in them to figure out >> which process is responsible. In any c

Re: CVS commit: src/tests/kernel

2013-02-19 Thread Joerg Sonnenberger
On Tue, Feb 19, 2013 at 02:45:43PM -0500, Julio Merino wrote: > On Tue, Feb 19, 2013 at 2:43 PM, Joerg Sonnenberger > wrote: > > On Tue, Feb 19, 2013 at 11:40:22AM -0800, Paul Goyette wrote: > >> Most of the printf's were already there before this round of > >> updates, and there's enough differen

Re: CVS commit: src/tests/kernel

2013-02-19 Thread Christos Zoulas
In article <20130219224428.5dd5717...@cvs.netbsd.org>, Paul Goyette wrote: >-=-=-=-=-=- > >+ const char *pipe_errmsg = "child: pipe write failed\n"; > > (void)unlink(lockfile); > >@@ -145,8 +146,8 @@ ATF_TC_BODY(randlock, tc) > switch (pid[i]) { > case 0: >

re: CVS commit: src/sys/kern

2013-02-19 Thread matthew green
> Module Name: src > Committed By: martin > Date: Tue Feb 19 11:20:17 UTC 2013 > > Modified Files: > src/sys/kern: subr_xcall.c > > Log Message: > Oops, accidently did not commit this part of pooka's change. please update the commit log to reference the original commit, rather th

re: CVS commit: src/tests

2013-02-19 Thread matthew green
> (Due to the way our directory creation and file lists work, I'm not > sure it's possible to just not "install" the empty directory. But > that'd be an alternative to this.) see eg, src/etc/mtree/NetBSD.dist.*. .mrg.