<HTML><BODY><br>
<blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;" class="mailru-blockquote">
<div id=""><div class="js-helper js-readmsg-msg">
<style type="text/css"></style>
<div id="style_13605112290000000457" class="mr_read__body">
<base target="_self" href="https://e.mail.ru/cgi-bin/">
<div id="style_13605112290000000457_BODY">Phileas Fogg <<a href="sentmsg?compose&To=phileas%2dfogg@mail.ru">phileas-fogg@mail.ru</a>> writes:<br><br>> Please ignore the previous patch to fix the PACA issue on PS3 arch.<br>> This is the correct one:<br>><br>> --- a/arch/powerpc/kernel/setup_64.c 2013-02-10 13:56:12.803855673 +0100<br>> +++ b/arch/powerpc/kernel/setup_64.c 2013-02-10 14:07:22.870561322 +0100<br>> @@ -186,6 +186,9 @@<br>> initialise_paca(&boot_paca, 0);<br>> setup_paca(&boot_paca);<br>> <br>> + /* Allow percpu accesses to "work" until we setup percpu data */<br>> + boot_paca.data_offset = 0;<br>> +<br>> /* Initialize lockdep early or else spinlocks will blow */<br>> lockdep_init();<br>> <br>><br><br>commit 466921c5a4669f4315528a25f9afd66601ce2c04 is done to fix the<br>lockdep related issue on ppc64. So this may need little bit more<br>explanation. So if we explicitly use boot_paca, do we still need the<br>changes in the above commit ?<br><br>-aneesh<br><br>_______________________________________________<br>Linuxppc-dev mailing list<br><a href="sentmsg?compose&To=Linuxppc%2ddev@lists.ozlabs.org">Linuxppc-dev@lists.ozlabs.org</a><br><a href="https://lists.ozlabs.org/listinfo/linuxppc-dev" target="_blank">https://lists.ozlabs.org/listinfo/linuxppc-dev</a><br></div>
<base target="_self" href="https://e.mail.ru/cgi-bin/">
</div>
</div>
</div>
</blockquote>
<br>Ok, here is the next PACA fix test.<br><br>I tested the following patch with Linux 3.8.0-rc7 on PS3 arch and still getting panics.<br><br>Patch:<br><br>--- arch/powerpc/kernel/setup_64.c.old 2013-02-10 19:34:53.787366191 +0100<br>+++ arch/powerpc/kernel/setup_64.c 2013-02-10 19:35:38.834035478 +0100<br>@@ -186,6 +186,9 @@<br> initialise_paca(&boot_paca, 0);<br> setup_paca(&boot_paca);<br> <br>+ /* Allow percpu accesses to "work" until we setup percpu data */<br>+ boot_paca.data_offset = 0;<br>+<br> /* Initialize lockdep early or else spinlocks will blow */<br> lockdep_init();<br> <br>@@ -208,8 +211,6 @@<br> <br> /* Fix up paca fields required for the boot cpu */<br> get_paca()->cpu_start = 1;<br>- /* Allow percpu accesses to "work" until we setup percpu data */<br>- get_paca()->data_offset = 0;<br> <br> /* Probe the machine type */<br> probe_machine();<br><br><br><br>It seems that 'boot_paca' and 'get_paca()' refer to different PACAs.<br><br>regards <br><br><br></BODY></HTML>