[lkml]   [2007]   [Dec]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RFC] [PATCH] A clean aEvgeniy pproach to writeout throttling
    On Monday 10 December 2007 02:47, Jens Axboe wrote:
    > Ehm, this patch is so broken it's not even funny - did you even
    > compile? You would have noticed the warning on request_queue_t,
    > surely. The big problem is the last hunk here though, how would that
    > work on stacked devices? Clue: ->bi_bdev is not const, it can change
    > after a call to ->make_request_fn().

    Such paranoia. Yes, the patch was compiled. Yes, the warning was
    slipped through. No, it is not substantive, and in fact was removed
    from another branch of our tree already.

    Ignoring the rhetoric, apparently you missed the line:

    + if (q && q->metric && !bio->bi_queue) {

    The prevents any reference ti bi_bdev after the intial call to
    generic_make_request. Thanks to Evgeniy for pointing out the need for
    this measure on the last go round.

    "So broken" is a gross exaggeration. Substantive comments welcome.


     \ /
      Last update: 2007-12-10 12:27    [W:0.027 / U:71.984 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site