linux-kernel - Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 10 Jun 2008 11:28:01 -0400
From:	"Martin K. Petersen" <martin.petersen@...cle.com>
To:	Jeff Moyer <jmoyer@...hat.com>
Cc:	"Martin K. Petersen" <martin.petersen@...cle.com>,
	linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org
Subject: Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support

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

Jeff> "Martin K. Petersen" <martin.petersen@...cle.com> writes:
>> 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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/