Messages in this thread | | | Date | Wed, 17 Oct 2007 02:53:42 +0200 | From | Björn Steinbrink <> | Subject | Re: arcmsr changelog differs from diffs |
| |
On 2007.10.17 00:07:19 +0200, Hans-Peter Jansen wrote: > Hi Jeff, > > while browsing through Linus' current check ins, I stumbled upon: > > [SCSI] arcmsr: irq handler fixes, cleanups, micro-opts: > > --8<-- > 488a5c8a9a3b67ae117784cd0d73bef53a73d57d > drivers/scsi/arcmsr/arcmsr_hba.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c > b/drivers/scsi/arcmsr/arcmsr_hba.c > index 7832a10..f4d2d52 100644 > --- a/drivers/scsi/arcmsr/arcmsr_hba.c > +++ b/drivers/scsi/arcmsr/arcmsr_hba.c > @@ -422,7 +422,7 @@ static int arcmsr_probe(struct pci_dev *pdev, > goto out_release_regions; > > error = request_irq(pdev->irq, arcmsr_do_interrupt, > - IRQF_SHARED, "arcmsr", acb); > + IRQF_SHARED, "arcmsr", acb); > if (error) > goto out_free_ccb_pool; > > -->8-- > > and: [SCSI] arcmsr: Fix hardware wait loops > > --8<-- > 24430458bb924e371ff894e26bfa9f73707f53fb > drivers/scsi/arcmsr/arcmsr_hba.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c > b/drivers/scsi/arcmsr/arcmsr_hba.c > index 50e1310..7832a10 100644 > --- a/drivers/scsi/arcmsr/arcmsr_hba.c > +++ b/drivers/scsi/arcmsr/arcmsr_hba.c > @@ -2092,8 +2092,10 @@ static void arcmsr_iop_reset(struct > AdapterControlBlock *acb) > if (atomic_read(&acb->ccboutstandingcount) != 0) { > /* talk to iop 331 outstanding command aborted */ > arcmsr_abort_allcmd(acb); > + > /* wait for 3 sec for all command aborted*/ > ssleep(3); > + > /* disable all outbound interrupt */ > intmask_org = arcmsr_disable_outbound_ints(acb); > /* clear all outbound posted Q */ > -->8-- > > where both changelogs differ significantly from the actual diffs, which both > are simple WS fixups and nothing else. Does qgit fools me here, or is > anything else wrong on my side?
Nothing wrong on your side. I took a look at the second one, and everything but the whitespace changes already found its way into Linus' tree via 1a4f550a09f89e3a15eff1971bc9db977571b9f6. One hunk of the original patch[1] was actually made redundant because the code was removed in that commit, so that's probably what James fixed (see full commit message).
And then, if I may guess, James probably just noticed that there were changes left and commited them (while they were now down to just the whitespace change), without checking what changes were actually left (no offense intended). At least I think that git wouldn't cripple the diff if the changes that James checked in were not already whitespace-only at the time he commited them, and the git history of his tree seems to agree.
Probably the other commit is similar.
Björn
[1] http://linux.derkeiler.com/Mailing-Lists/Kernel/2007-07/msg11957.html - 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/
| |