Re: Linux v2.6.9... (compile stats)

From: viro
Date: Wed Oct 20 2004 - 21:40:26 EST


On Wed, Oct 20, 2004 at 09:59:47PM -0400, Jeff Garzik wrote:
> >Hmm... It misses a bunch of easy stuff, actually (tons of casts to void *
> >from what used to be unsigned long and is void __iomem * with your patch).
>
> feel free to send a delta :)

Will do.

> >I don't see where you handle PIO stuff, though - no ioport_map() _or_
> >pci_iomap() in sight.
>
> Correct, that part doesn't exist yet. grep in the above quoted text for
> "* map/unap" for the to-do list.
>
> The mapping of the PIO PCI BARs requires independently mapping at least
> 5 (but varies from controller to controller) IO port ranges, and
> tracking those mappings in a coherent manner.

IDGI. Why do you insist on releasing these guys in library code? Even
with iomem case you are creating mappings in driver code, so the reverse
should also be done there...
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/