Re: CVS commit: src/external/cddl/osnet

2011-01-14 Thread Robert Elz
Date:Sat, 15 Jan 2011 14:36:50 +1100 From:matthew green Message-ID: <399.1295062...@splode.eterna.com.au> | > You also need to debug code with the same compilation options as you | > will use for any final builds - otherwise you can get some very | > nasty surpr

re: CVS commit: src/external/cddl/osnet

2011-01-14 Thread matthew green
> > I would like to have zfs build with -g -O0 built by default, > > this is wip code and there is high probability that someone > > will find some problem with it, therefore I would like to have > > debugging symbols already builtin. > > You really don't want to be using -O0, ever. > I certainly

Build break - CVS commit: src/lib/libc/time

2011-01-14 Thread Paul Goyette
Module Name:src Committed By: christos Date: Fri Jan 14 23:35:07 UTC 2011 Modified Files: src/lib/libc/time: Makefile localtime.c zdump.c zic.c Log Message: don't depend on integer oveflow wrapping to detect it. From Paul Eggert in the tz mailing list. This appears to h

Re: CVS commit: src/external/cddl/osnet

2011-01-14 Thread David Laight
On Fri, Jan 14, 2011 at 10:18:29AM +0100, haad wrote: > > I would like to have zfs build with -g -O0 built by default, > this is wip code and there is high probability that someone > will find some problem with it, therefore I would like to have > debugging symbols already builtin. You really don

Re: CVS commit: src/lib/libp2k

2011-01-14 Thread Antti Kantee
On Fri Jan 14 2011 at 14:49:36 +, David Holland wrote: > > Do you have some relevant scenarios where "fses set flags in > > componentname"? > > For example, ufs_lookup sets ISWHITEOUT, which is later checked by > ufs_mkdir and ufs_create and others. ... which is used only with unions which

Re: CVS commit: src/lib/libp2k

2011-01-14 Thread David Holland
On Fri, Jan 14, 2011 at 11:22:33AM +0200, Antti Kantee wrote: > > > Remove the componentname-saving code since it was addressing > > > SAVENAME/HASBUF/SAVESTART and they don't exist anymore (and the > > > removed code didn't compile on nb5). > > > > Unfortunately, you still need that code for

Re: CVS commit: src/distrib

2011-01-14 Thread haad
On Fri, Jan 14, 2011 at 11:26 AM, Izumi Tsutsui wrote: > Module Name:    src > Committed By:   tsutsui > Date:           Fri Jan 14 10:26:38 UTC 2011 > > Modified Files: >        src/distrib/acorn26/instkernel: list >        src/distrib/acorn32/ramdisk: list >        src/distrib/alpha/instkernel/r

Re: CVS commit: src/lib/libp2k

2011-01-14 Thread Antti Kantee
On Fri Jan 14 2011 at 08:42:45 +, David Holland wrote: > On Fri, Jan 07, 2011 at 11:15:31AM +, Antti Kantee wrote: > > Modified Files: > >src/lib/libp2k: p2k.c > > > > Log Message: > > Remove the componentname-saving code since it was addressing > > SAVENAME/HASBUF/SAVESTART and

Re: CVS commit: src/external/cddl/osnet

2011-01-14 Thread haad
On Fri, Jan 14, 2011 at 1:01 AM, matthew green wrote: > >> Module Name:  src >> Committed By: haad >> Date:         Thu Jan 13 23:18:24 UTC 2011 >> >> Modified Files: >>       src/external/cddl/osnet: Makefile.zfs >>       src/external/cddl/osnet/lib/libavl: Makefile >>       src/external/cddl/osn

Re: CVS commit: src

2011-01-14 Thread David Holland
On Mon, Jan 10, 2011 at 07:45:23PM +, David Laight wrote: > > As far as I can tell it only uses the usecount to decide whether the > > vnode is still active or not, so adding 1 temporarily should be > > sufficient if the goal is to have it not disappear. > > > > OTOH, it seems to me that

Re: CVS commit: src/lib/libp2k

2011-01-14 Thread David Holland
On Fri, Jan 07, 2011 at 11:15:31AM +, Antti Kantee wrote: > Modified Files: > src/lib/libp2k: p2k.c > > Log Message: > Remove the componentname-saving code since it was addressing > SAVENAME/HASBUF/SAVESTART and they don't exist anymore (and the > removed code didn't compile on nb5