On Thu, Oct 18, 2007 at 10:26:58AM -0400, Alan Stern wrote: > > Even that's still pretty foul (and i have my grave doubts about using > > stringify in that manner). It's definitely past my bedtime now ... but > > a hack like that could save us another 8 bytes on x86-64. > > This may seem heretical, but I doubt there's much point to saving a few > bytes if it's going to incur this much work and confusion. Rearranging > the order of structure members is fine, it's easy, it won't bother > anybody. But this sort of thing is just beyond the bounds of reason. That's a valid point of view. But we're just 24 bytes shy of going down from 6 cachelines to 5. I don't have performance numbers yet, but the memory savings are significant; we switch from 10 cmds per 4k slab page to 12 cmds per 4k slab page. By the way, after my patch 4/4 getting rid of transfersize, we're down to 256 bytes on i386. That's 16 cmds per 4k slab page, so we alloc 25% fewer pages to scsi_cmnds. That's a *huge* win. -- Intel are signing my paycheques ... these opinions are still mine "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
