From owner-freebsd-fs@FreeBSD.ORG Fri May 9 18:32:29 2003 Return-Path: Delivered-To: freebsd-fs@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 931) id 0DE8537B401; Fri, 9 May 2003 18:32:29 -0700 (PDT) Date: Fri, 9 May 2003 20:32:29 -0500 From: Juli Mallett To: Lukas Ertl Message-ID: <20030509203228.A62797@FreeBSD.org> References: <15610.1052520021@critter.freebsd.dk> <20030510004537.H638@korben.in.tern> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <20030510004537.H638@korben.in.tern>; from l.ertl@univie.ac.at on Sat, May 10, 2003 at 12:47:26AM +0200 X-Title: Code Maven X-Towel: Yes X-Negacore: Yes X-Authentication-Warning: localhost: juli pwned teh intarweb cc: freebsd-fs@freebsd.org cc: Poul-Henning Kamp Subject: Re: newfs: useless/bogus check if new last block can be accessed? X-BeenThere: freebsd-fs@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Filesystems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 10 May 2003 01:32:29 -0000 * Lukas Ertl [ Date: 2003-05-09 ] [ w.r.t. Re: newfs: useless/bogus check if new last block can be accessed? ] > On Sat, 10 May 2003, Poul-Henning Kamp wrote: > > > In message <20030510002107.T638@korben.in.tern>, Lukas Ertl writes: > > > > >I don't think it does any harm to keep it (WRT to getting a working > > >filesystem), but since I don't see how it would signal an error (and this > > >is why I posted the question) I'd vote for removing the check. > > > > > >And while we're here: shouldn't wtfs() actually give a return value and > > >not be just static void? (In terms of: "be a good programmer and check the > > >return values of your syscalls...") I think it's called often enough in > > >newfs to qualify for a check :-) > > > > Get in touch with jmallett@freebsd.org who has been actively trying to > > improve the UFS/FFS tools for some time, it sounds like the two of you > > are on the same page :-) > > :-) > > Ok, although I guess she's on this list anyway, I'm CC'ing her and hope > she's interested if I can come up with some fixes... I at one point had some warnx calls (at minimum, since things tend to work and it's better to let someone shoot their foot maybe than use errx, cause at that point, it's a bit late.) and so on. I didn't ever get them into CVS as I was working on general libufs-ification at the time of newfs, and then tracking down why that broke things (*blush*), and I didn't want to put in too many new failure cases I could be blamed for :) Do a bunch of testing with errx, maybe, see if things blow up, and if so, maybe make it advisory (e.g. warnx) and track down the real root of the problem. When I was discussing some generalised API for accessing members of the sblock (which went nowhere due to over-engineering, and too much faith...er, yeah, it went nowhere), one of the things I wanted to do was have a "naive" flag as part of an (undeveloped) generalised flags (external, not like MINE_) interface, which would let libufs do the exploding, if things went wrong. But that's beyond what you're talking about. Do some testing, maybe some cleanups along what you mentioned, I'll be glad to see it done, as long as things still work :) Thanx, juli. -- juli mallett. email: jmallett@freebsd.org; efnet: juli;