Date: Mon, 07 Jan 2002 10:26:40 -0800 (PST) From: John Baldwin <jhb@FreeBSD.org> To: Matthew Dillon <dillon@apollo.backplane.com> Cc: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: RE: cvs commit: src/lib/libdisk chunk.c libdisk.h Message-ID: <XFMail.020107102640.jhb@FreeBSD.org> In-Reply-To: <200201070828.g078S4M59083@apollo.backplane.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 07-Jan-02 Matthew Dillon wrote: >:> Modified files: >:> lib/libdisk chunk.c libdisk.h >:> Log: >:> Modify Delete_Chunk() into Delete_Chunk2() which can take a flags >:> argument. Leave a compatibility shim for Delete_Chunk(). >:> >:> Implement DELCHUNK_RECOVER flag so sysinstall can ask libdisk >:> to recover space when deleting a chunk. >: >:Why not implement a Merge_Chunk() that takes two chunks and tries to delete >:the >:second and move its space into the first. This would seem to be a cleaner >:change in the API. >: >:-- >: >:John Baldwin <jhb@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/ >:"Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ > > If you had suggested this a month ago when I (A) was doing most of > this work and (B) had time and (C) we weren't 8? days away from > an RC or release, then maybe. Now... you'll have to wait until > Jordan gets his stuff in after the release or someone else decides > to work on the code (instead of just complain about it). Hum, since you just committed this to current I figured it wasn't in the patches you had already committed to current. :) This was a comment on a change you made to libdisk API, not on the sysinstall change as a whole. > -Matt -- John Baldwin <jhb@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?XFMail.020107102640.jhb>