On 2012-09-06 16:34, Kent Overstreet wrote: > Reusing bios is something that's been highly frowned upon in the past, > but driver code keeps doing it anyways. If it's going to happen anyways, > we should provide a generic method. > > This'll help with getting rid of bi_destructor - drivers/block/pktcdvd.c > was open coding it, by doing a bio_init() and resetting bi_destructor. > > This required reordering struct bio, but the block layer is not yet > nearly fast enough for any cacheline effects to matter here. That's an odd and misplaced comment. Was just doing testing today at 5M IOPS, and even years back we've had cache effects for O_DIRECT in higher speed setups. That said, we haven't done cache analysis in a long time. So moving members around isn't necessarily a huge deal. Lastly, this isn't a great commit message for other reasons. Anyone can see that it moves members around. It'd be a lot better to explain _why_ it is reordering the struct. BTW, I looked over the rest of the patches, and it looks OK to me. However, you didn't CC me on the full series, so I'm missing bits 3 and 5. -- Jens Axboe -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel