On Mon, Oct 22, 2007 at 11:52:51PM +0100, Alan Cox wrote:On Mon, 22 Oct 2007 16:47:07 -0500
Matt Mackall <mpm@xxxxxxxxxxx> wrote:
On Mon, Oct 22, 2007 at 05:21:30PM -0400, Jeff Garzik wrote:Is that really a credible space issue ?Alan Cox wrote:It'll mean m-1 scatterlists fit on a slab.Why can't we just make the list one item longer than the entry count andCertainly seems safer than the current "let's run off the end of the list if anything bad happens" setup... And I do not think allocating n+1 scatterlist entries will have much of a negative impact.
stick a NULL on the end of it like normal people ?
Yes. Especially if m is 2 or 1. A scatterlist on 64-bit x86 looks like
it takes 32 bytes, which means 128 elements fit on a page. One more
spills - ouch!