On Fri, Oct 06, 2006 at 03:15:15PM -0400, Jeff Garzik wrote: > Matthew Wilcox wrote: > >Also, pci_set_mwi() will fail if the cache line > >size is 0, so we don't need to check that ourselves any more. > > NAK, not true on all arches. sparc64 at least presumes that the > firmware DTRT with cacheline size, which hurts us now given this tulip patch
How does it hurt us? int pcibios_prep_mwi(struct pci_dev *dev) { /* We set correct PCI_CACHE_LINE_SIZE register values for every * device probed on this platform. So there is nothing to check * and this always succeeds. */ return 0; } If Dave's wrong about that, it hurts him, not us ;-) It's still not necessary for the Tulip driver to check. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html