Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 2 Feb 2008 09:40:13 +0900
From:      Pyun YongHyeon <pyunyh@gmail.com>
To:        Poul-Henning Kamp <phk@FreeBSD.org>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/share/man/man9 mbuf.9 src/sys/compat/ndis kern_ndis.c src/sys/dev/cxgb cxgb_sge.c src/sys/dev/cxgb/ulp/tom cxgb_cpl_socket.c src/sys/dev/hatm if_hatm_intr.c src/sys/dev/iscsi/initiator isc_soc.c src/sys/dev/lge if_lge.c ...
Message-ID:  <20080202004013.GA17592@cdnetworks.co.kr>
In-Reply-To: <200802011936.m11JaR2n084955@repoman.freebsd.org>
References:  <200802011936.m11JaR2n084955@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Feb 01, 2008 at 07:36:27PM +0000, Poul-Henning Kamp wrote:
 > phk         2008-02-01 19:36:27 UTC
 > 
 >   FreeBSD src repository
 > 
 >   Modified files:
 >     share/man/man9       mbuf.9 
 >     sys/compat/ndis      kern_ndis.c 
 >     sys/dev/cxgb         cxgb_sge.c 
 >     sys/dev/cxgb/ulp/tom cxgb_cpl_socket.c 
 >     sys/dev/hatm         if_hatm_intr.c 
 >     sys/dev/iscsi/initiator isc_soc.c 
 >     sys/dev/lge          if_lge.c 
 >     sys/dev/msk          if_msk.c 
 >     sys/dev/nfe          if_nfe.c 
 >     sys/dev/patm         if_patm_rx.c 
 >     sys/dev/sk           if_sk.c 
 >     sys/dev/ti           if_ti.c 
 >     sys/dev/wpi          if_wpi.c 
 >     sys/kern             kern_mbuf.c uipc_cow.c uipc_mbuf.c 
 >                          uipc_syscalls.c 
 >     sys/pci              if_wb.c 
 >     sys/sys              mbuf.h param.h 
 >   Log:
 >   Give MEXTADD() another argument to make both void pointers to the
 >   free function controlable, instead of passing the KVA of the buffer
 >   storage as the first argument.
 >   
 >   Fix all conventional users of the API to pass the KVA of the buffer
 >   as the first argument, to make this a no-op commit.
 >   

Because physically contiguous 9K jumbos are available I should
have nuked local jumbo allocators used in msk(4)/nfe(4) and sk(4).
If that happened you didn't need to touch these drivers.

Sorry for that and thanks for your work.

 >   Likely break the only non-convetional user of the API, after informing
 >   the relevant committer.
 >   
 >   Update the mbuf(9) manual page, which was already out of sync on
 >   this point.
 >   
 >   Bump __FreeBSD_version to 800016 as there is no way to tell how
 >   many arguments a CPP macro needs any other way.
 >   
 >   This paves the way for giving sendfile(9) a way to wait for the
 >   passed storage to have been accessed before returning.
 >   
 >   This does not affect the memory layout or size of mbufs.
 >   
 >   Parental oversight by:  sam and rwatson.
 >   
 >   No MFC is anticipated.

[...]

-- 
Regards,
Pyun YongHyeon



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080202004013.GA17592>