Re: [PATCH 1/1] block: CFQ refcounting fix

From: Jens Axboe (axboe_at_suse.de)
Date: 08/31/05

  • Next message: Nilesh Agrawal: "Re: tty problem"
    Date:	Wed, 31 Aug 2005 17:50:24 +0200
    To: Brian King <brking@us.ibm.com>
    
    

    On Wed, Aug 31 2005, Brian King wrote:
    > diff -puN drivers/block/cfq-iosched.c~cfq_refcnt_fix drivers/block/cfq-iosched.c
    > --- linux-2.6/drivers/block/cfq-iosched.c~cfq_refcnt_fix 2005-08-30 17:26:55.000000000 -0500
    > +++ linux-2.6-bjking1/drivers/block/cfq-iosched.c 2005-08-31 08:48:30.000000000 -0500
    > @@ -2260,8 +2260,6 @@ static void cfq_put_cfqd(struct cfq_data
    > if (!atomic_dec_and_test(&cfqd->ref))
    > return;
    >
    > - blk_put_queue(q);
    > -
    > cfq_shutdown_timer_wq(cfqd);
    > q->elevator->elevator_data = NULL;
    >
    > @@ -2318,7 +2316,6 @@ static int cfq_init_queue(request_queue_
    > e->elevator_data = cfqd;
    >
    > cfqd->queue = q;
    > - atomic_inc(&q->refcnt);
    >
    > cfqd->max_queued = q->nr_requests / 4;
    > q->nr_batching = cfq_queued;
    > _

    That looks better. I'll add this to my outgoing queue, thanks!

    -- 
    Jens Axboe
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at  http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at  http://www.tux.org/lkml/
    

  • Next message: Nilesh Agrawal: "Re: tty problem"

    Relevant Pages

    • Re: I request inclusion of reiser4 in the mainline kernel
      ... On Tue, Sep 20 2005, Hans Reiser wrote: ... >>Seeing as you are the one that is apparently bothered by the misnomer, ... Jens Axboe ... To unsubscribe from this list: send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [PATCH] AHCI PCI MSI support, update 1
      ... LOC: 671016 671636 ... Jens Axboe - ... To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ ...
      (Linux-Kernel)
    • Re: AS spin lock bugs
      ... On Thu, Nov 13 2003, Jens Axboe wrote: ... BTW, this looks bogus. ... Why do you need any locking there? ... To unsubscribe from this list: send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [PATCH] [IDE] remove ide_dma_queued_* ops from ide_hwif_t
      ... On Tue, Feb 03 2004, Bartlomiej Zolnierkiewicz wrote: ... Jens Axboe ... To unsubscribe from this list: send the line "unsubscribe linux-kernel" in ... Please read the FAQ at http://www.tux.org/lkml/ ...
      (Linux-Kernel)
    • Re: [PATCH] blk queue io tracing support
      ... On Wed, Aug 24 2005, Nathan Scott wrote: ... Jens Axboe ... To unsubscribe from this list: send the line "unsubscribe linux-kernel" in ... Please read the FAQ at http://www.tux.org/lkml/ ...
      (Linux-Kernel)