All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Lord <liml@rtr.ca>
To: Jens Axboe <jens.axboe@oracle.com>
Cc: Mark Lord <lkml@rtr.ca>, Matthew Wilcox <matthew@wil.cx>,
	IDE/ATA development list <linux-ide@vger.kernel.org>,
	Linux Kernel <linux-kernel@vger.kernel.org>,
	linux-scsi <linux-scsi@vger.kernel.org>,
	Neil Brown <neilb@suse.de>
Subject: Re: QUEUE_FLAG_CLUSTER: not working in 2.6.24 ?
Date: Thu, 13 Dec 2007 14:53:21 -0500	[thread overview]
Message-ID: <47618DB1.6030408@rtr.ca> (raw)
In-Reply-To: <20071213192633.GD10104@kernel.dk>

Jens Axboe wrote:
> On Thu, Dec 13 2007, Mark Lord wrote:
>> Matthew Wilcox wrote:
>>> On Thu, Dec 13, 2007 at 01:48:18PM -0500, Mark Lord wrote:
>>>> Problem confirmed.  2.6.23.8 regularly generates segments up to 64KB for 
>>>> libata,
>>>> but 2.6.24 uses only 4KB segments and a *few* 8KB segments.
>>> Just a suspicion ... could this be slab vs slub?  ie check your configs
>>> are the same / similar between the two kernels.
>> ..
>>
>> Mmmm.. a good thought, that one.
>> But I just rechecked, and both have CONFIG_SLAB=y
>>
>> My guess is that something got changed around when Jens
>> reworked the block layer for 2.6.24.
>> I'm going to dig around in there now.
> 
> I didn't rework the block layer for 2.6.24 :-). The core block layer
> changes since 2.6.23 are:
> 
> - Support for empty barriers. Not a likely candidate.
> - Shared tag queue fixes. Totally unlikely.
> - sg chaining support. Not likely.
> - The bio changes from Neil. Of the bunch, the most likely suspects in
>   this area, since it changes some of the code involved with merges and
>   blk_rq_map_sg().
> - Lots of simple stuff, again very unlikely.
> 
> Anyway, it sounds odd for this to be a block layer problem if you do see
> occasional segments being merged. So it sounds more like the input data
> having changed.
> 
> Why not just bisect it?
..

CC'ing Neil Brown.


      parent reply	other threads:[~2007-12-13 19:53 UTC|newest]

Thread overview: 90+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-13 18:36 QUEUE_FLAG_CLUSTER: not working in 2.6.24 ? Mark Lord
2007-12-13 18:37 ` Mark Lord
2007-12-13 18:42   ` Matthew Wilcox
2007-12-13 18:46     ` James Bottomley
2007-12-13 18:48   ` Mark Lord
2007-12-13 18:53     ` Matthew Wilcox
2007-12-13 19:03       ` Mark Lord
2007-12-13 19:26         ` Jens Axboe
2007-12-13 19:30           ` Mark Lord
2007-12-13 19:32             ` Mark Lord
2007-12-13 19:39               ` Jens Axboe
2007-12-13 19:42                 ` Mark Lord
2007-12-13 19:53                   ` Jens Axboe
2007-12-13 19:59                     ` Mark Lord
2007-12-13 20:05                       ` Jens Axboe
2007-12-13 20:02                     ` Jens Axboe
2007-12-13 20:06                       ` Mark Lord
2007-12-13 20:09                         ` Jens Axboe
2007-12-13 20:14                           ` Mark Lord
2007-12-13 20:18                             ` Mark Lord
2007-12-13 20:21                             ` Jens Axboe
2007-12-13 22:02                           ` Andrew Morton
2007-12-13 22:02                             ` Andrew Morton
2007-12-13 22:15                             ` James Bottomley
2007-12-13 22:15                               ` James Bottomley
2007-12-13 22:29                               ` Andrew Morton
2007-12-13 22:29                                 ` Andrew Morton
2007-12-13 22:33                                 ` Mark Lord
2007-12-13 22:33                                   ` Mark Lord
2007-12-13 23:13                                   ` Mark Lord
2007-12-13 23:13                                     ` Mark Lord
2007-12-14  0:05                                     ` Mark Lord
2007-12-14  0:05                                       ` Mark Lord
2007-12-14  0:30                                       ` Mark Lord
2007-12-14  0:30                                         ` Mark Lord
2007-12-14  0:37                                         ` Andrew Morton
2007-12-14  0:37                                           ` Andrew Morton
2007-12-14  0:42                                           ` Mark Lord
2007-12-14  0:42                                             ` Mark Lord
2007-12-14  0:46                                             ` [PATCH] fix page_alloc for larger I/O segments (improved) Mark Lord
2007-12-14  0:46                                               ` Mark Lord
2007-12-14  0:57                                               ` James Bottomley
2007-12-14  0:57                                                 ` James Bottomley
2007-12-14  1:11                                                 ` Andrew Morton
2007-12-14  1:11                                                   ` Andrew Morton
2007-12-14  2:23                                                   ` Mark Lord
2007-12-14  2:23                                                     ` Mark Lord
2007-12-14  2:23                                                     ` Mark Lord
2007-12-14 17:42                                               ` Mel Gorman
2007-12-14 17:42                                                 ` Mel Gorman
2007-12-14 18:07                                                 ` Mark Lord
2007-12-14 18:07                                                   ` Mark Lord
2007-12-16 21:56                                                   ` Mel Gorman
2007-12-16 21:56                                                     ` Mel Gorman
2007-12-14 18:13                                                 ` Matthew Wilcox
2007-12-14 18:13                                                   ` Matthew Wilcox
2007-12-14 18:30                                                   ` Mark Lord
2007-12-14 18:30                                                     ` Mark Lord
2007-12-20 22:37                                                   ` Matthew Wilcox
2007-12-20 22:37                                                     ` Matthew Wilcox
2007-12-14  0:47                                             ` QUEUE_FLAG_CLUSTER: not working in 2.6.24 ? Mark Lord
2007-12-14  0:47                                               ` Mark Lord
2007-12-14 11:50                                           ` Mel Gorman
2007-12-14 11:50                                             ` Mel Gorman
2007-12-14 13:57                                             ` Mark Lord
2007-12-14 13:57                                               ` Mark Lord
2007-12-14  0:40                                         ` [PATCH] fix page_alloc for larger I/O segments Mark Lord
2007-12-14  0:40                                           ` Mark Lord
2007-12-14  1:03                                           ` Andrew Morton
2007-12-14  1:03                                             ` Andrew Morton
2007-12-14  4:00                                             ` Matthew Wilcox
2007-12-14  4:00                                               ` Matthew Wilcox
2007-12-15  1:09                                 ` QUEUE_FLAG_CLUSTER: not working in 2.6.24 ? Mel Gorman
2007-12-15  1:09                                   ` Mel Gorman
2007-12-15  2:02                                   ` Andrew Morton
2007-12-15  2:02                                     ` Andrew Morton
2007-12-15  5:55                                     ` Matt Mackall
2007-12-15  5:55                                       ` Matt Mackall
2007-12-16 21:55                                     ` Mel Gorman
2007-12-16 21:55                                       ` Mel Gorman
2007-12-17 19:24                                       ` Randy Dunlap
2007-12-17 19:24                                         ` Randy Dunlap
2007-12-18  2:42                                         ` Matt Mackall
2007-12-18  2:42                                           ` Matt Mackall
2007-12-13 22:17                             ` Jens Axboe
2007-12-13 22:17                               ` Jens Axboe
2007-12-13 22:02                           ` VM allocates pages in reverse order again Matthew Wilcox
2007-12-13 22:02                             ` Matthew Wilcox
2007-12-13 19:37             ` QUEUE_FLAG_CLUSTER: not working in 2.6.24 ? Jens Axboe
2007-12-13 19:53           ` Mark Lord [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=47618DB1.6030408@rtr.ca \
    --to=liml@rtr.ca \
    --cc=jens.axboe@oracle.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=lkml@rtr.ca \
    --cc=matthew@wil.cx \
    --cc=neilb@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.