Messages in this thread | | | Date | Fri, 6 Aug 2004 14:18:36 +0100 | From | Christoph Hellwig <> | Subject | Re: Altix I/O code reorganization |
| |
On Wed, Aug 04, 2004 at 03:14:08PM -0500, Pat Gefre wrote: > o added new hardware support > o code cleanup (typedefs, include files, etc.) > o simplified the directory structure (all files were arch/ia64/sn/io/ > are now under arch/ia64/sn/ioif/) > o code size reduced by >50% > o major reorg of the code itself > o copyright updates
Yikes, this is truely horrible. First your patch ordering doesn't make any sense, with just the first patch applied the system won't work at all. Please submit a series of _small_ patches going from A to B keeping the code working everywhere inbetween.
Your new directory structure is very bad. Just stick all files into arch/ia64/sn/io/ instead of adding subdirectories for often just a single file.
Now to the contents:
002-pci-fixups: you're adding tons of non-standard SAL calls for who knows what. In fact this pretty much looks like you're just moving the existing crappy code into the prom so the bad Linux guys can't complain about it anymore. Please switch to the standard ACPI PCI probing mechanism all other IA64 machines support and you can get rid of all that. You're duplicating the kernel's PCI to PCI bridge support, with the normal IA64 pci code it would just work..
003-pci-support: The PCI DMA implementation is still ubercomplicated. See the PCI DMA code I sent you long ago. Of the code in pci_extensions.c only two are actually used in the kernel (snia_pcibr_rrb_alloc and snia_pcidev_endian_set), please remove the unused other ones.
004-pci-bridge_drivers: You still have code dealing with all kinds of PCIIO_ and PCIBR_ flags that will never be set through the Linux interfaces. Again see the DMA mapping code I sent you.
006-bte: Please merge bte_error.c into the existing bte.c
007-io-hub-provider: tio_provider and hub_provider have exactly the same methods, no need to keep the xtalk_provider_t abstraction at all
008-kdb-support-funtions: kdb isn't in mainline, please add the two files to the kdb patch instead
- 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/
| |