lkml.org 
[lkml]   [2012]   [Sep]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v10 4/8] block: Add bio_reset()
    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



    \
     
     \ /
      Last update: 2012-09-07 04:21    [W:2.290 / U:0.176 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site