On Saturday, September 22, 2007 10:23 AM, James Bottomley wrote: > > OK, I thought I'd wait here for the breakout, but in the meantime I > tried to compile the first five patches, but they don't: > > CC [M] drivers/message/fusion/mptscsih.o > drivers/message/fusion/mptscsih.c: In function 'mptscsih_qcmd': > drivers/message/fusion/mptscsih.c:1357: error: 'MPT_SCSI_HOST' has no > member named 'resetPending' > drivers/message/fusion/mptscsih.c: In function 'mptscsih_TMHandler': > drivers/message/fusion/mptscsih.c:1554: error: 'MPT_ADAPTER' has no > member named 'diagLock' > ... Its not going to compile if you didn't pull down all 6 patchs associated to "error recovery improvements". Those six patchs were divided by sources. Meaning patch 4 was mptbase.[h,c] changes, then patch5 was mptctl.[c,h] changes, and so on. So the changes assoicated for mptscsich.c are in patch 8. Since you didn't apply the 8th patch, offcourse your going to get the errrors in mptscish.c. All patchs 4-9 need to go togeather, as mentioned in the first patch 0. > > The series can't be applied in this form, because if git bisect steps > into the middle of this, the compile will break (and hence the > bisection) and a lot of people will say a lot of nasty things. > > A patch series really needs to be one patch per logical change (with > other peoples' patches broken out) in a form that is separately > compilable for each patch. Additionally, with a separate > change log and > summary (i.e. not four patches all saying "mpt fusion: error recovery > improvements, and synchronizing internal commands"). > I understand, and thats been going on for the past couple months, starting with Sathya's patchs early August. The changes associated with the rewrite of internal commands, and errror recovery is rather difficult to seperate due to many depandancies and new structure changes, and my concerned risk of what small steps could introduce more issues, so IMO, the least risk was jump starting you to the customer and factory tested code. > I'll back all of this out; can you resend the series conforming to the > above request? > The first three patchs did conform that, which are: 1) adding usage of shost_priv and removing all the typecasting 2) Fixing sparse warnings 3) locking down ScsiLookup Youve rejected the error recovery patchs, which is fair enough. - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
