cvs commit: src/sys/geom geom.h geom_io.c

Pawel Jakub Dawidek pjd at FreeBSD.org
Tue Jun 6 06:44:16 UTC 2006


On Tue, Jun 06, 2006 at 06:50:29AM +0200, Poul-Henning Kamp wrote:
+> In message <200606052113.k55LDMks055707 at repoman.freebsd.org>, Pawel Jakub Dawid
+> ek writes:
+> 
+> >  Add g_duplicate_bio() function which does the same thing what g_clone_bio()
+> >  is doing, but g_duplicate_bio() allocates new bio with M_WAITOK flag.
+> 
+> First, this sounds wrong to me from an overall memory management point
+> of view.
+> 
+> Second, g_clone_bio_wait() had been a better name.

I thought so too when you added g_alloc_bio() function (a M_WAITOK
version of g_new_bio())... :)

-- 
Pawel Jakub Dawidek                       http://www.wheel.pl
pjd at FreeBSD.org                           http://www.FreeBSD.org
FreeBSD committer                         Am I Evil? Yes, I Am!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20060606/c50968b5/attachment.pgp


More information about the cvs-src mailing list