garrett_damore@tadpole.com wrote:

> The "general" solution is to make if_ath_pci.c and if_ath_cardbus.c
> store the bus_handle_t in an OS-specific handle, but then that means
> that all architectures get penalized by another level of pointer
> indirection.  (Making Atheros' "tag" really be a pointer to bus_handle_t
> instead of a bus_handle_t itself.)

Do you have any benchmark to measure the performance penalty
by pointer indirection? I doubt it's measurable.

>  Architecturally this is clean, but
> it seems a poor choice for performance.

There is a guideline "only clean code" in src/doc/TODO
so I don't think you could get consensus at the point.. ;-p

> Bus handles (or opaque access handles for the Atheros HAL) are currently
> stored in void *.  I cannot change that bit.    I'm trying to make it
> possible to address the problem of making this work well for NetBSD in
> the normal case (where the types are pointers anyway), while also
> allowing sparc bogosity to work as well.  See above for more explanation.

I agree that it's sometimes problematic to use non-pointer type
structure for bus_space_handle_t on implementing real MD drivers,
but it's worse to introduce some loophole like
__BUS_SPACE_HANDLE_IS_STRUCT in MI APIs. It's still better
to define some new API to handle the opaque type as christos said.
---
Izumi Tsutsui