From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ron Mercer Date: Tue, 14 Oct 2008 02:53:29 +0000 Subject: Re: [PATCH] 2.6.27-rc9 QGLE make unsupported page size explicit compile failure Message-Id: <20081014025329.GA25397@susedev.qlogic.org> List-Id: References: <20081014015455.GA25385@susedev.qlogic.org> <20081013.190345.20202604.davem@davemloft.net> <20081014021902.GY25780@parisc-linux.org> <20081013.193153.203217396.davem@davemloft.net> In-Reply-To: <20081013.193153.203217396.davem@davemloft.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: David Miller Cc: matthew@wil.cx, grundler@parisc-linux.org, linux-driver@qlogic.com, netdev@vger.kernel.org, linux-ia64@vger.kernel.org On Mon, Oct 13, 2008 at 07:31:53PM -0700, David Miller wrote: > From: Matthew Wilcox > Date: Mon, 13 Oct 2008 20:19:02 -0600 > > > On Mon, Oct 13, 2008 at 07:03:45PM -0700, David Miller wrote: > > > From: Ron Mercer > > > Date: Mon, 13 Oct 2008 18:54:55 -0700 > > > > > > > This ASIC does support all page sizes. For 4k and 8k page size the TX control block needs an external scatter gather list. For page sizes larger than 8k the max frags is satisfied by the original TX control block. > > > > > > I'm not applying this. I _explicitly_ said I would not apply a > > > solution like this. > > > > > > I waited for you guys all day for this?!?! > > > > > > We need to find a way to make this a Kconfig dependency. > > > > > > Build failures are absolutely unacceptable. > > > > Um, Dave, I think you misread his patch. > > > > All it does it make the '64k case' the 'every other size case'. There's > > no build failure with Ron's patch. > > Thanks for the correction, I'll look at this again :) My explanation might not have been very good. What I was trying to say is that the macro in question determines the size of a scatter/gathter list that is external to the TX descriptor. The size is based on MAX_SKB_FRAGS and PAGE_SIZE. When PAGE_SIZE is larger than 8k I don't need the external scatter/gather list because all frags will fit in the native TX descriptor.