its_alloc_tables() - with all BASER marked none

its_alloc_tables() - with all BASER marked none

[Mario Smarduch]

If its_init()/its_probe()/its_alloc_tables() finds all GITS_BASER type none, it
continues with ITS initialization.

Is there a reason to continue?

Started to look through this code recently - little confused here.

Thanks,
  mario

Re: its_alloc_tables() - with all BASER marked none

[Marc Zyngier]

On 15/10/15 02:30, Mario Smarduch wrote:
> If its_init()/its_probe()/its_alloc_tables() finds all GITS_BASER type none, it
> continues with ITS initialization.
> 
> Is there a reason to continue?

Of course. There is nothing that *mandates* the ITS to request memory
from the operating system. The HW could perfectly come with its own
memory and not need anything at all.

> Started to look through this code recently - little confused here.

Only the beginning! ;-)

	M.
-- 
Jazz is not dead. It just smells funny...

Re: its_alloc_tables() - with all BASER marked none

[Mario Smarduch]

On 10/15/2015 12:50 AM, Marc Zyngier wrote:
> On 15/10/15 02:30, Mario Smarduch wrote:
>> If its_init()/its_probe()/its_alloc_tables() finds all GITS_BASER type none, it
>> continues with ITS initialization.
>>
>> Is there a reason to continue?
> 
> Of course. There is nothing that *mandates* the ITS to request memory
> from the operating system. The HW could perfectly come with its own
> memory and not need anything at all.
> 
>> Started to look through this code recently - little confused here.
> 
> Only the beginning! ;-)

Well at this time the beginning of console freeze close to  "ITS: using cache
flushing for cmd queue". Poked around CBASER and noticed Inner cache bits are
updated but not sticky so it got me wondering.

Appears like I need to look elsewhere, like maybe my environment.

Thanks for clarifying.

- Mario

> 
> 	M.
> 

Re: its_alloc_tables() - with all BASER marked none

[Marc Zyngier]

On 15/10/15 18:08, Mario Smarduch wrote:
> 
> 
> On 10/15/2015 12:50 AM, Marc Zyngier wrote:
>> On 15/10/15 02:30, Mario Smarduch wrote:
>>> If its_init()/its_probe()/its_alloc_tables() finds all GITS_BASER type none, it
>>> continues with ITS initialization.
>>>
>>> Is there a reason to continue?
>>
>> Of course. There is nothing that *mandates* the ITS to request memory
>> from the operating system. The HW could perfectly come with its own
>> memory and not need anything at all.
>>
>>> Started to look through this code recently - little confused here.
>>
>> Only the beginning! ;-)
> 
> Well at this time the beginning of console freeze close to  "ITS: using cache
> flushing for cmd queue". Poked around CBASER and noticed Inner cache bits are
> updated but not sticky so it got me wondering.
> 
> Appears like I need to look elsewhere, like maybe my environment.

Out of curiosity, is that on a FastModel? Or on some actual HW?

	M.
-- 
Jazz is not dead. It just smells funny...

Re: its_alloc_tables() - with all BASER marked none

[Mario Smarduch]

On 10/15/2015 10:10 AM, Marc Zyngier wrote:
> On 15/10/15 18:08, Mario Smarduch wrote:
>>
>>
>> On 10/15/2015 12:50 AM, Marc Zyngier wrote:
>>> On 15/10/15 02:30, Mario Smarduch wrote:
>>>> If its_init()/its_probe()/its_alloc_tables() finds all GITS_BASER type none, it
>>>> continues with ITS initialization.
>>>>
>>>> Is there a reason to continue?
>>>
>>> Of course. There is nothing that *mandates* the ITS to request memory
>>> from the operating system. The HW could perfectly come with its own
>>> memory and not need anything at all.
>>>
>>>> Started to look through this code recently - little confused here.
>>>
>>> Only the beginning! ;-)
>>
>> Well at this time the beginning of console freeze close to  "ITS: using cache
>> flushing for cmd queue". Poked around CBASER and noticed Inner cache bits are
>> updated but not sticky so it got me wondering.
>>
>> Appears like I need to look elsewhere, like maybe my environment.
> 
> Out of curiosity, is that on a FastModel? Or on some actual HW?

FastModel, license ran out on FVP.
> 
> 	M.
>