>>>>> "Jeff" == Jeff Moyer <jmoyer@redhat.com> writes:

Jeff> "Martin K. Petersen" <martin.petersen@oracle.com> writes:
quoted text
>> Another post of my block I/O data integrity patches.  This kit goes
>> on top of the scsi_data_buffer and sd.h cleanups I posted earlier
>> today.

Jeff> Pointers to archives would have been appreciated.  I can't, for
Jeff> the life of me, find these.

  http://marc.info/?l=linux-scsi&m=121272302931588&w=2
  http://marc.info/?l=linux-scsi&m=121278031605941&w=2
  http://marc.info/?l=linux-scsi&m=121302438515260&w=2
  http://marc.info/?l=linux-scsi&m=121278067906564&w=2


Jeff> Thanks for all of the great documentation.  It would be good to
Jeff> include some instructions on how one would test this, and what
Jeff> testing you performed.

modprobe scsi_debug dix=199 dif=1 guard=1 dev_size_mb=1024 num_parts=1

I'm testing with XFS and btrfs.  Generally doing kernel builds, etc.
ext2/3 are still problematic because they modify pages in flight.


Jeff> I set out to try your changes, but ran into some problems.
Jeff> First, this patch set didn't apply cleanly to a git checkout.

I generally track Linus closely so it must be because of the patches
you were missing.  

You can grab my patch stack here.  It's always in sync with the hg
repo:

    http://oss.oracle.com/~mkp/patches/


Jeff> block/blk-core.c: In function 'generic_make_request':
Jeff> include/linux/bio.h:469: sorry, unimplemented: inlining failed
Jeff> in call to 'bio_i ntegrity_enabled': function body not available
Jeff> block/blk-core.c:1388: sorry, unimplemented: called from here
Jeff> make[1]: *** [block/blk-core.o] Error 1 make: *** [block] Error
Jeff> 2

Odd.  Which compiler are you using?  Compiles just fine for me on both
EL5 and FC9.

Judging from the error I'm guessing it's objecting to the inlining.
Tried to work around it.  Please pull, update and let me know whether
that did the trick.

-- 
Martin K. Petersen	Oracle Linux Engineering

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