[Nouveau] Is it necessary to parse the VBIOS init table when executable is false?

[Ben Skeggs]

On 20 Nov 2014 01:31, "李金华" <goodqt at outlook.com> wrote:
>
> Hello everyone,
>
> The code is at /core/subdev/bios/init.c.
> When the executable is 0,these functions that parse the VBIOS init table
seems to do nothing except the parsing.
Correct.  We do not handle every opcode, only the ones that have been
observed so far.  The purpose is to be able to test whether we will be able
to fully execute the table, to avoid failing in a critical path (such as
resume from suspend) where we can't recover.

Ben.

And in my opinion,init_exec_force does nothing,too!
>
> Then I tried to test it on my computer,when the VBIOS parser tries to
change a MMIO register or call a function of struct nouveau_devinit (such
as devinit->pll_set,etc.),it will print a message into the kernel log.
However,I cannot see any message about this in the kernel log!
>
> Finally,I decided to just return when executable is zero. Then I found
out the driver also works perfectly.
>
> So I asked the question here,thanks for your help.
> _______________________________________________
> Nouveau mailing list
> Nouveau at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/nouveau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20141120/5786d304/attachment.html>