Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756500AbYFJP3F (ORCPT ); Tue, 10 Jun 2008 11:29:05 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754521AbYFJP2m (ORCPT ); Tue, 10 Jun 2008 11:28:42 -0400 Received: from rgminet01.oracle.com ([148.87.113.118]:23969 "EHLO rgminet01.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754304AbYFJP2k (ORCPT ); Tue, 10 Jun 2008 11:28:40 -0400 To: Jeff Moyer Cc: "Martin K. Petersen" , linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org Subject: Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support From: "Martin K. Petersen" Organization: Oracle References: Date: Tue, 10 Jun 2008 11:28:01 -0400 In-Reply-To: (Jeff Moyer's message of "Tue\, 10 Jun 2008 10\:41\:45 -0400") Message-ID: User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Brightmail-Tracker: AAAAAQAAAAI= X-Brightmail-Tracker: AAAAAQAAAAI= X-Whitelist: TRUE X-Whitelist: TRUE Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2201 Lines: 60 >>>>> "Jeff" == Jeff Moyer writes: Jeff> "Martin K. Petersen" 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/