Re: [RFC] Allow m_dup() to use JUMBO clusters

2014-07-23 Thread John-Mark Gurney
Rick Macklem wrote this message on Tue, Jul 08, 2014 at 16:10 -0400: > I tried: > m = m_getjcl(M_NOWAIT..M_JUMPAGESIZE); > if (m == NULL) >m = getjcl(M_WAITOK..MCLBYTES); > when I was experimenting with MJUMPAGESIZE clusters for NFS and what happened > was the thread looped

Re: [RFC] Allow m_dup() to use JUMBO clusters

2014-07-08 Thread Rick Macklem
John-Mark Gurney wrote: > Hans Petter Selasky wrote this message on Mon, Jul 07, 2014 at 10:12 > +0200: > > I'm asking for some input on the attached m_dup() patch, so that > > existing functionality or dependencies are not broken. The > > background > > for the change is to allow m_dup() to defrag

Re: [RFC] Allow m_dup() to use JUMBO clusters

2014-07-08 Thread Rick Macklem
Hans Petter Selasky wrote: > > > > Hi, > > > > Would it be better if my patch used the PAGE_SIZE clusters instead > > of > > the 16K ones? Then it should not be affected by memory > > defragmentation. > > Thanks for shedding some light into this area? > > Well, I ran into the threads stuck on "btal

Re: [RFC] Allow m_dup() to use JUMBO clusters

2014-07-08 Thread Hans Petter Selasky
Hi, Would it be better if my patch used the PAGE_SIZE clusters instead of the 16K ones? Then it should not be affected by memory defragmentation. Thanks for shedding some light into this area? --HPS Hi, Updated patch attached. --HPS === sys/kern/uipc_mbuf.c

Re: [RFC] Allow m_dup() to use JUMBO clusters

2014-07-07 Thread Hans Petter Selasky
On 07/08/14 04:14, Yonghyeon PYUN wrote: On Mon, Jul 07, 2014 at 10:12:07AM +0200, Hans Petter Selasky wrote: Hi, I'm asking for some input on the attached m_dup() patch, so that existing functionality or dependencies are not broken. The background for the change is to allow m_dup() to defrag l

Re: [RFC] Allow m_dup() to use JUMBO clusters

2014-07-07 Thread Yonghyeon PYUN
On Mon, Jul 07, 2014 at 10:12:07AM +0200, Hans Petter Selasky wrote: > Hi, > > I'm asking for some input on the attached m_dup() patch, so that > existing functionality or dependencies are not broken. The background > for the change is to allow m_dup() to defrag long mbuf chains that > doesn't

Re: [RFC] Allow m_dup() to use JUMBO clusters

2014-07-07 Thread John-Mark Gurney
Hans Petter Selasky wrote this message on Mon, Jul 07, 2014 at 10:12 +0200: > I'm asking for some input on the attached m_dup() patch, so that > existing functionality or dependencies are not broken. The background > for the change is to allow m_dup() to defrag long mbuf chains that > doesn't fi

Re: [RFC] Allow m_dup() to use JUMBO clusters

2014-07-07 Thread Rick Macklem
Hans Petter Selasky wrote: > Hi, > > I'm asking for some input on the attached m_dup() patch, so that > existing functionality or dependencies are not broken. The background > for the change is to allow m_dup() to defrag long mbuf chains that > doesn't fit into a specific hardware's scatter gather

[RFC] Allow m_dup() to use JUMBO clusters

2014-07-07 Thread Hans Petter Selasky
Hi, I'm asking for some input on the attached m_dup() patch, so that existing functionality or dependencies are not broken. The background for the change is to allow m_dup() to defrag long mbuf chains that doesn't fit into a specific hardware's scatter gather entries, typically when doing TSO