Re: CVS commit: src/sys/dev/pci

[Soren Jorvang <soren%jorvang.dk@localhost>]

On 23/07/2013, at 11.04, Martin Husemann <martin%duskware.de@localhost> wrote:
> On Tue, Jul 23, 2013 at 10:09:28AM +0200, Soren Jorvang wrote:
>> The pre-autoconf console code creates a bus_space handle (not a
>> subregion of anything) for the console com, which conflicts with puc.c
>> later trying to allocate the larger handle that the console is a
>> subregion (but not a bus_space_subregion) of.
> 
> An alternative hack would be to unmap the original com console mapping
> and update it with a subregion from the puc BAR mapping.

That should probably be part of the proper solution that I mentioned since 
pci_mapreg_map() only allows mapping whole BAR regions, but for it to be not 
just another x86 hack, the interaction between puc/com_puc/com needs to be 
thought out to avoid either killing the console during the child handoff or 
putting more comcn knowledge into puc.c.


-- 
Soren