From owner-freebsd-current Thu Jan 23 15:38:14 2003 Delivered-To: freebsd-current@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 931) id B902237B401; Thu, 23 Jan 2003 15:38:12 -0800 (PST) Date: Thu, 23 Jan 2003 15:38:12 -0800 From: Juli Mallett To: Nate Lawson Cc: current@FreeBSD.org Subject: Re: Test this! Patch to make newfs(8) use libufs. Message-ID: <20030123153811.A66302@FreeBSD.org> References: <20030123142924.A61481@FreeBSD.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: ; from nate@root.org on Thu, Jan 23, 2003 at 03:17:38PM -0800 Organisation: The FreeBSD Project X-Alternate-Addresses: , , , , X-Towel: Yes X-LiveJournal: flata, jmallett X-Negacore: Yes Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG * De: Nate Lawson [ Data: 2003-01-23 ] [ Subjecte: Re: Test this! Patch to make newfs(8) use libufs. ] > On Thu, 23 Jan 2003, Juli Mallett wrote: > > * De: Nate Lawson [ Data: 2003-01-23 ] > > [ Subjecte: Re: Test this! Patch to make newfs(8) use libufs. ] > > > On Thu, 23 Jan 2003, Juli Mallett wrote: > > > > Thanks to Ruslan's reminder that tunefs now uses libufs and tunefs is > > > > one of the crunched programs, I realised that I really needed to make > > > > newfs(8) use libufs. To show off that it *can* help us reduce space, > > > > a good bit in some cases. > > > > > > Good to see this. Does libufs do an fsync() in bwrite() or do you need to > > > do that manually in place of the previous calls to wtfsflush()? > > > > It doesn't. Do you think it should? I'd rather do that than litter the > > newfs code. > > Not needed. On second look, I see you removed the wc[] caching and fall > back to just write(). (BTW, what's the performance difference with your > patch?) If newfs doesn't complete successfully due to a system crash, it > doesn't matter if data written was flushed to disk, just run it again. *nod*. Not to mention that I don't think you need it for writing to a raw device? As for performance, I don't know. Test it and find out? :) -- Juli Mallett AIM: BSDFlata -- IRC: juli on EFnet. OpenDarwin, Mono, FreeBSD Developer. ircd-hybrid Developer, EFnet addict. FreeBSD on MIPS-Anything on FreeBSD. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message