lkml.org 
[lkml]   [2003]   [Dec]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Another dm and bio problem with 2.6.0
On Mon, Dec 22 2003, Mark Haverkamp wrote:
> This fixes a problem similar to the patch I submitted on 11/20
>
> http://marc.theaimsgroup.com/?l=linux-kernel&m=106936439707962&w=2
>
> In this case, though, the result is an:
>
> "Incorrect number of segments after building list" message.
>
> The macro __BVEC_START assumes a bi_idx of zero when the dm code can
> submit a bio with a non-zero bi_idx.
> The code has been tested on an 8 way / 8gb OSDL STP machine with a 197G
> lvm volume running dbt2 test.
>
> ===== include/linux/bio.h 1.34 vs edited =====
> --- 1.34/include/linux/bio.h Sun Sep 21 14:50:33 2003
> +++ edited/include/linux/bio.h Wed Dec 17 07:17:53 2003
> @@ -162,7 +162,7 @@
> */
>
> #define __BVEC_END(bio) bio_iovec_idx((bio), (bio)->bi_vcnt - 1)
> -#define __BVEC_START(bio) bio_iovec_idx((bio), 0)
> +#define __BVEC_START(bio) bio_iovec_idx((bio), (bio)->bi_idx)
> #define BIOVEC_PHYS_MERGEABLE(vec1, vec2) \
> ((bvec_to_phys((vec1)) + (vec1)->bv_len) == bvec_to_phys((vec2)))
> #define BIOVEC_VIRT_MERGEABLE(vec1, vec2) \

Must have been some nasty debugging, good job. The patch is correct.

--
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: 2005-03-22 13:59    [W:0.028 / U:0.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site