Re: [dm-devel] Re: dm: bounce_pfn limit added

2007-10-31 Thread Alasdair G Kergon
On Wed, Oct 31, 2007 at 05:00:16PM -0500, Kiyoshi Ueda wrote: > How about the case that other dm device is stacked on the dm device? > (e.g. dm-linear over dm-multipath over i2o with bounce_pfn=64GB, and > the multipath table is changed to i2o with bounce_pfn=1GB.) Let's not broaden the pro

Re: [dm-devel] Re: dm: bounce_pfn limit added

2007-10-31 Thread Kiyoshi Ueda
Hi, On Wed, 31 Oct 2007 08:36:01 +0100, Hannes Reinecke <[EMAIL PROTECTED]> wrote: > Vasily Averin wrote: > > Alasdair G Kergon wrote: > >> So currently we treat bounce_pfn as a property that does not need to be > >> propagated through the stack. > >> > >> But is that the right approach? > >> - Is

Re: [dm-devel] Re: dm: bounce_pfn limit added

2007-10-30 Thread Hannes Reinecke
Vasily Averin wrote: > Alasdair G Kergon wrote: >> So currently we treat bounce_pfn as a property that does not need to be >> propagated through the stack. >> >> But is that the right approach? >> - Is there a blk_queue_bounce() missing either from dm or elsewhere? >> (And BTW can the bio_alloc()

Re: [dm-devel] Re: dm: bounce_pfn limit added

2007-10-30 Thread Vasily Averin
Alasdair G Kergon wrote: > So currently we treat bounce_pfn as a property that does not need to be > propagated through the stack. > > But is that the right approach? > - Is there a blk_queue_bounce() missing either from dm or elsewhere? > (And BTW can the bio_alloc() that lurks within lead to d

Re: [dm-devel] Re: dm: bounce_pfn limit added

2007-10-30 Thread Alasdair G Kergon
On Wed, Oct 31, 2007 at 02:01:33AM +, Alasdair G Kergon wrote: > What if you swap in alternative dm targets, e.g. if it's linear, > try multipath (round-robin, one path)? And try using md instead of dm - does that also show the problem? (md takes a similar stance to dm on this I believe.) A

Re: [dm-devel] Re: dm: bounce_pfn limit added

2007-10-30 Thread Alasdair G Kergon
On Tue, Oct 30, 2007 at 11:26:17PM +, Alasdair G Kergon wrote: > ) DM doesn't need to bounce bio's on its own, but the block layer defaults > ) to that in blk_queue_make_request(). The lower level drivers should > ) bounce ios themselves, that is what they need to do if not layered below > ) dm

Re: [dm-devel] Re: dm: bounce_pfn limit added

2007-10-30 Thread Alasdair G Kergon
On Tue, Oct 30, 2007 at 01:11:38PM -0700, Andrew Morton wrote: > On Mon, 29 Oct 2007 09:31:39 +0300 > Vasily Averin <[EMAIL PROTECTED]> wrote: > > Device mapper uses its own bounce_pfn that may differ from one on underlying > > device. In that way dm can build incorrect requests that contain sg ele