[OpenBIOS] [PATCH 1/8] Rename ofmem_arch_early_map_pages() to ofmem_arch_map_pages().

Mark Cave-Ayland mark.cave-ayland at ilande.co.uk
Fri Apr 13 18:03:08 CEST 2012


On 10/04/12 18:45, Igor Kovalenko wrote:

>> Well, it's more interesting than that. For SPARC, when you map the memory it
>> is first added to the translation_t linked list. The TLB miss handlers
>> simply iterate over this list in order to find the mapping before
>> continuing. Therefore when creating a mapping, since the translation_t list
>> already contains the entry created by ofmem_map_page_range(), all the
>> ofmem_arch_map_pages() function has to do is update any existing TLB
>> entries.
>>
>
> The "early" variant was intended to perform one-time injection of
> mapped pages that were acquired before c source code is entered so it
> was not safe to call common mapping routine yet. I was afraid to
> implement mapping code in assembly.

I think I see this now. Okay - I think it would be possible to either 
merge this into the existing OFMEM code or refactor it to make use of 
the existing functions.

It looks like I definitely need to produce a v2 of this patch, so I 
won't commit it so far and submit a revised version shortly.


Many thanks,

Mark.



More information about the OpenBIOS mailing list