Correction. The problem occurs when 8 bios of size 512 with 1 bvec each 
all share the same page. I made a bad assumption previously.

-Kai Meyer

On 12/01/2011 10:49 AM, Kai Meyer wrote:

I want to be able to copy data into a struct bio *, so I use
bio_for_each_segment to loop through each bvec, like so:

void some_function(struct bio *bio, char *some_data) {
      struct bio_vec *bvec;
      int i;
      unsigned int bio_so_far = 0;
      bio_for_each_segment(bvec, bio, i) {
          char *bio_buffer = __bio_kmap_atomic(bio, i, KM_USER0);
          memcpy(bio_buffer, some_data + bio_so_far, bvec->bv_len);
          __bio_kunmap_atomic(bio, KM_USER0);
          bio_so_far += bvec->bv_len;
      }
}

There's lots more to the function, but this is basically the distilled
version with out any extra stuff.

What I'm finding is that when the bio has multiple bvecs that share the
same page, only the first bvec's data actually gets copied back up to
user-space, the rest is garbage or null (meaning, what was there
already). For instance, I see a lot of bios from vfat that are 4096
bytes long but are comprised of 8 bvecs that are 512 bytes long that all
have an offset to the same page.

I've tried doing just one kmap_atomic on the page by keeping track of
what the last page I kmap'ed was, but that didn't fix the problem either.

Any documentation or high level explanation of kmap/kunmap or other
ideas to try are welcome.

-Kai Meyer

_______________________________________________
Kernelnewbies mailing list
Kernelnewbies at kernelnewbies.org
http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies