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

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Jeff Moyer <jmoyer@...>
Cc: Martin K. Petersen <martin.petersen@...>, <linux-kernel@...>, <linux-scsi@...>
Date: Tuesday, June 10, 2008 - 4:47 pm

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

Jeff> So, is it safe to say that the library routines for
Jeff> integrity-aware file systems have not been tested at all?
Jeff> Specifically, I'm talking about: bio_integrity_tag_size
Jeff> bio_integrity_set_tag bio_integrity_get_tag

I have not tried using them from within a filesystem, if that's what
you mean.  But I have attached random strings to bios and read them
back later.


Jeff> gcc (GCC) 4.1.2 20071124 (Red Hat 4.1.2-41)

Ok, I'm trying to chase down a 5.2 box to figure out what the problem
is.  Maybe I'll just move that function to the header file.


Jeff> I did a new clone (just to be sure I got your change) and I get
Jeff> the same problem.  I also can't see the changeset in the log, so
Jeff> are you sure you pushed it?

Yup, it's there.


Jeff> I got rid of the inline in the definition in bio.h.  The .c file
Jeff> didn't define the function as inline, so I didn't have to change
Jeff> it.  It seems to be building now.

The problem is that your gcc is unhappy about the fact that the
inlined function is defined elsewhere.  The gcc info page said only
declare it inline in the header and not the declaration.  The change I
pushed removed inline from the .c file.  But that didn't help.

-- 
Martin K. Petersen	Oracle Linux Engineering

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0 of 7] Block/SCSI Data Integrity Support, Martin K. Petersen, (Sat Jun 7, 12:55 am)
Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support, Jeff Moyer, (Tue Jun 10, 10:41 am)
Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support, Martin K. Petersen, (Tue Jun 10, 11:28 am)
Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support, Mike Snitzer, (Thu Jul 17, 9:55 am)
Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support, Martin K. Petersen, (Thu Jul 17, 11:35 am)
Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support, Jeff Moyer, (Tue Jun 10, 2:49 pm)
Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support, Martin K. Petersen, (Tue Jun 10, 4:47 pm)
Re: [PATCH 0 of 7] Block/SCSI Data Integrity Support, Jeff Moyer, (Tue Jun 10, 4:53 pm)
[PATCH 7 of 7] Support for SCSI disk (SBC) Data Integrity Fi..., Martin K. Petersen, (Sat Jun 7, 12:55 am)
[PATCH 5 of 7] block: Block/request layer data integrity sup..., Martin K. Petersen, (Sat Jun 7, 12:55 am)
Re: [PATCH 5 of 7] block: Block/request layer data integrity..., Martin K. Petersen, (Mon Jun 9, 11:06 am)
[PATCH 4 of 7] block: bio data integrity support, Martin K. Petersen, (Sat Jun 7, 12:55 am)
Re: [PATCH 4 of 7] block: bio data integrity support, Jeff Moyer, (Tue Jun 10, 4:52 pm)
Re: [PATCH 4 of 7] block: bio data integrity support, Martin K. Petersen, (Wed Jun 11, 12:05 am)
Re: [PATCH 4 of 7] block: bio data integrity support, Jeff Moyer, (Wed Jun 11, 1:41 pm)
Re: [PATCH 4 of 7] block: bio data integrity support, Monakhov Dmitri, (Sat Jun 7, 10:45 am)
Re: [PATCH 4 of 7] block: bio data integrity support, Martin K. Petersen, (Mon Jun 9, 11:05 am)
[PATCH 6 of 7] scsi: Support devices with protection informa..., Martin K. Petersen, (Sat Jun 7, 12:55 am)
[PATCH 1 of 7] lib: Add support for the T10 Data Integrity F..., Martin K. Petersen, (Sat Jun 7, 12:55 am)
[PATCH 2 of 7] block: Globalize bio_set and bio_vec_slab, Martin K. Petersen, (Sat Jun 7, 12:55 am)
[PATCH 3 of 7] block: Find bio sector offset given idx and o..., Martin K. Petersen, (Sat Jun 7, 12:55 am)
Re: [PATCH 3 of 7] block: Find bio sector offset given idx a..., Martin K. Petersen, (Mon Jun 9, 12:15 pm)