Messages in this thread | | | Date | Wed, 18 May 2005 10:24:47 -0700 | From | Matthew Dobson <> | Subject | Re: 2.6.12-rc4-mm2 build failure |
| |
Christoph Lameter wrote: > On Tue, 17 May 2005, Matthew Dobson wrote: > > >>Not a big fan of this patch. It's not wrong, per-se, but it just doesn't >>sit right with me. asm-generic/topology.h should be a fallback file for >>the arches that just want some sort of sane UP/SMP defaults. The better >>(IMHO) solution is this patch. Builds fine on PPC64. > > > And what happens if yet another function needs to be added for all arches? > Then ppc64 will break again and you will fix it again?
I think you explained it well yourself. If another function needs to be added for ALL ARCHES, then ALL ARCHES will need to add the function. In most cases there is no single function that is both CORRECT and GENERIC across all arches. The way that i386, ia64, ppc64, etc. will map PCI Buses to Nodes (for instance) will NOT be the same. Anyone who adds a new topology function has the responsibility of 1) making sure it works for all arches which support topology, or 2) getting the arch maintainers involved and helping them make sure it works for all arches.
New topology functions don't really get added all that often. We've got the basics (CPU, Mem, I/O Buses, Nodes) mapped in various ways, so there shouldn't be tons of new functions added. If someone wants to add a new function, it's their responsibility to make sure that it doesn't break anyone's arch.
Regardless, I highly doubt Linus or Andrew will pick up a patch for a new topology function if it breaks some arch.
-Matt - 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/
| |