lkml.org 
[lkml]   [2007]   [Mar]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: 2.6.21-rc2-mm2
On Tue, Mar 06 2007, J.A. Magallón wrote:
> On Tue, 6 Mar 2007 00:44:08 -0800, Andrew Morton <akpm@linux-foundation.org> wrote:
>
> >
> > Temporarily at
> >
> > http://userweb.kernel.org/~akpm/2.6.21-rc2-mm2/
> >
>
> Does this include a fix for the NFS problem ?
>
> BTW, I have in my kernel a patch like this below, isn't it needed ?
> Original thread:
>
> http://marc.theaimsgroup.com/?l=linux-kernel&m=117189051227292&w=2
>
> --- a/block/ll_rw_blk.c
> +++ b/block/ll_rw_blk.c
> @@ -2919,14 +2919,14 @@ static int __make_request(request_queue_
> */
> blk_queue_bounce(q, &bio);
>
> + spin_lock_irq(q->queue_lock);
> /*
> * Check if we can merge with the plugged list before grabbing
> * any locks.
> */
> if (!check_plug_merge(q, ioc, bio))
> - goto out;
> + goto out_unlock;
>
> - spin_lock_irq(q->queue_lock);
> el_ret = elv_merge(q, &req, bio);
> if (el_ret == ELEVATOR_BACK_MERGE) {
> if (bio_attempt_back_merge(q, req, bio)) {
> @@ -2984,7 +2984,6 @@ out_unlock:
> list_add_tail(&req->queuelist, &ioc->plugged_list);
> }
>
> -out:
> return 0;
>
> end_io_eopnotsupp:

No it's not, plus andrew didn't include git-block in this release so
you'd have a hard time even applying it. The above patch would also
eliminate 50% of the win of per-process plugging, if you go and grab the
queue lock anyway.

--
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/

\
 
 \ /
  Last update: 2007-03-06 12:19    [W:0.513 / U:0.420 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site