Re: linux-next: libata/avr32 build failure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Randy Dunlap wrote:
On Fri, 04 Apr 2008 16:23:53 -0400 Jeff Garzik wrote:

Randy Dunlap wrote:
Jeff Garzik wrote:
Randy Dunlap wrote:
On Fri, 04 Apr 2008 04:06:56 -0400 Jeff Garzik wrote:

Stephen Rothwell wrote:
Hi Jeff,

For a while the linux-next build for avr32 defconfig (and others) has
been failing like this:

drivers/ata/libata-core.c:127: error: 'ata_pci_default_filter' undeclared here (not in a function)

It seems to be because the declaration of ata_pci_default_filter is
protected by CONFIG_PCI but the usage above is protected by CONFIG_ATA
only.

Also reported by Randy Dunlap (on March 26) and Haavard Skinnemoen (on
March 29).
fixed being pushed right now (patch from Tejun fixed it)
Some fixes other than what was in today's (April 04)
[git patches] libata fixes?

I hope there is more.  I don't see how that fixes this error.
Nor does gcc see how, after I apply that patch.
It wasn't sent in #upstream-fixes, it was applied to #upstream...

Is that where you are looking?
I'm just trying to build linux-next with or without today's "libata fixes".
ATA with CONFIG_PCI=n still has build errors, as reported above.
OK, that's expected.

Uh, that's not good.  What does it mean?

(a) linux-next is not pulling the correct branch of your git tree
or
(b) linux-next just won't work with libata
or
(c) we have too many trees in the forest
or
something else?  (if so, what?)

It means you are inside the window between unfixed linux-next and tomorrow's fixed linux-next, and that you attempted to seek the proper fix in the wrong location.

So the next time linux-next pulls from libata, it should be fixed.

Or, if you don't want to wait, you should get the fix from the location noted (#upstream branch), because the fix was not among those sent to Linus yesterday.

	Jeff


--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux