<div dir="ltr">Can you rebase and resent the patch please then?<div><br></div><div>Cheers,</div><div>Ross</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 6 January 2017 at 02:40, Bai, Haiqing (Wind River) <span dir="ltr">&lt;<a href="mailto:haiqing.bai@windriver.com" target="_blank">haiqing.bai@windriver.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div class="m_-918250190295239798WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:&quot;Calibri&quot;,&quot;sans-serif&quot;;color:#1f497d">Hi, Ross:<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:&quot;Calibri&quot;,&quot;sans-serif&quot;;color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:&quot;Calibri&quot;,&quot;sans-serif&quot;;color:#1f497d">      I clone the branch (</span>poky-contrib:ross/mut<span style="font-size:11.0pt;font-family:&quot;Calibri&quot;,&quot;sans-serif&quot;;color:#1f497d">) and did’t find the fix
 in 2.0.14,  so this fix is still required.  Thanks<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:&quot;Calibri&quot;,&quot;sans-serif&quot;;color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:&quot;Calibri&quot;,&quot;sans-serif&quot;;color:#1f497d">B,R<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:&quot;Calibri&quot;,&quot;sans-serif&quot;;color:#1f497d">Haiqing Bai<u></u><u></u></span></p>
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:&quot;Tahoma&quot;,&quot;sans-serif&quot;">From:</span></b><span style="font-size:10.0pt;font-family:&quot;Tahoma&quot;,&quot;sans-serif&quot;"> Burton, Ross [mailto:<a href="mailto:ross.burton@intel.com" target="_blank">ross.burton@intel.com</a>]
<br>
<b>Sent:</b> 2017</span><span lang="ZH-CN" style="font-size:10.0pt;font-family:宋体">年</span><span style="font-size:10.0pt;font-family:&quot;Tahoma&quot;,&quot;sans-serif&quot;">1</span><span lang="ZH-CN" style="font-size:10.0pt;font-family:宋体">月</span><span style="font-size:10.0pt;font-family:&quot;Tahoma&quot;,&quot;sans-serif&quot;">4</span><span lang="ZH-CN" style="font-size:10.0pt;font-family:宋体">日</span><span style="font-size:10.0pt;font-family:&quot;Tahoma&quot;,&quot;sans-serif&quot;">
 22:32<br>
<b>To:</b> Bai, Haiqing<br>
<b>Cc:</b> OE-core<br>
<b>Subject:</b> Re: [OE-core] [PATCH] kexec: ARM: fix align issue of add_buffer_phys_virt() for LPAE kernel.<u></u><u></u></span></p><div><div class="h5">
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">An upgrade to 2.0.14 was recently posted on the list (and is now in poky-contrib:ross/mut), can you verify that this patch is still required with that release and rebase?<u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Ross<u></u><u></u></p>
</div>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">On 3 January 2017 at 09:05, Haiqing Bai &lt;<a href="mailto:Haiqing.Bai@windriver.com" target="_blank">Haiqing.Bai@windriver.com</a>&gt; wrote:<u></u><u></u></p>
<p class="MsoNormal">If LPAE is enabled, 3 level page table is used and the &#39;SECTION_SIZE&#39;<br>
is (1&lt;&lt;21), so add_buffer_phys_virt() should align to (1 &lt;&lt; 21).<br>
<br>
Signed-off-by: Haiqing Bai &lt;<a href="mailto:Haiqing.Bai@windriver.com" target="_blank">Haiqing.Bai@windriver.com</a>&gt;<br>
---<br>
 ...-ARM-fix-add_buffer_phys_<wbr>virt-align-issue.patch | 52 ++++++++++++++++++++++<br>
 meta/recipes-kernel/kexec/<a href="http://kexec-tools_2.0.12.bb" target="_blank">kex<wbr>ec-tools_2.0.12.bb</a>    |  1 +<br>
 2 files changed, 53 insertions(+)<br>
 create mode 100644 meta/recipes-kernel/kexec/<wbr>kexec-tools/kexec-ARM-fix-add_<wbr>buffer_phys_virt-align-issue.<wbr>patch<br>
<br>
diff --git a/meta/recipes-kernel/kexec/<wbr>kexec-tools/kexec-ARM-fix-add_<wbr>buffer_phys_virt-align-issue.<wbr>patch b/meta/recipes-kernel/kexec/<wbr>kexec-tools/kexec-ARM-fix-add_<wbr>buffer_phys_virt-align-issue.<wbr>patch<br>
new file mode 100644<br>
index 0000000..7d1f95e<br>
--- /dev/null<br>
+++ b/meta/recipes-kernel/kexec/<wbr>kexec-tools/kexec-ARM-fix-add_<wbr>buffer_phys_virt-align-issue.<wbr>patch<br>
@@ -0,0 +1,52 @@<br>
+From 380019f68e19ac863cf32dc7ff6784<wbr>e8fe1d751f Mon Sep 17 00:00:00 2001<br>
+From: Haiqing Bai &lt;<a href="mailto:Haiqing.Bai@windriver.com" target="_blank">Haiqing.Bai@windriver.com</a>&gt;<br>
+Date: Mon, 19 Dec 2016 14:52:02 +0800<br>
+Subject: [PATCH] kexec: ARM: Fix add_buffer_phys_virt() align issue.<br>
+<br>
+When &quot;CONFIG_ARM_LPAE&quot; is enabled,3 level page table<br>
+is used by MMU, the &quot;SECTION_SIZE&quot; is defined with<br>
+(1 &lt;&lt; 21), but &#39;add_buffer_phys_virt()&#39; hardcode this<br>
+to (1 &lt;&lt; 20).<br>
+<br>
+Upstream-Status: Pending<br>
+<br>
+<a href="mailto:Suggested-By%3Afredrik.markstrom@gmail.com" target="_blank">Suggested-By:fredrik.<wbr>markstrom@gmail.com</a><br>
+Signed-off-by: Haiqing Bai &lt;<a href="mailto:Haiqing.Bai@windriver.com" target="_blank">Haiqing.Bai@windriver.com</a>&gt;<br>
+---<br>
+ kexec/arch/arm/crashdump-arm.c | 5 ++++-<br>
+ 1 file changed, 4 insertions(+), 1 deletion(-)<br>
+<br>
+diff --git a/kexec/arch/arm/crashdump-<wbr>arm.c b/kexec/arch/arm/crashdump-<wbr>arm.c<br>
+index 38a1401..e41b7fb 100644<br>
+--- a/kexec/arch/arm/crashdump-<wbr>arm.c<br>
++++ b/kexec/arch/arm/crashdump-<wbr>arm.c<br>
+@@ -271,6 +271,7 @@ int load_crashdump_segments(struct kexec_info *info, char *mod_cmdline)<br>
+       void *buf;<br>
+       int err;<br>
+       int last_ranges;<br>
++      unsigned short align_bit_shift = 20;<br>
+<br>
+       /*<br>
+        * First fetch all the memory (RAM) ranges that we are going to pass to<br>
+@@ -295,6 +296,7 @@ int load_crashdump_segments(struct kexec_info *info, char *mod_cmdline)<br>
+<br>
+               /* for support LPAE enabled kernel*/<br>
+               elf_info.class = ELFCLASS64;<br>
++              align_bit_shift = 21;<br>
+<br>
+               err = crash_create_elf64_headers(<wbr>info, &amp;elf_info,<br>
+                                        usablemem_rgns.ranges,<br>
+@@ -315,8 +317,9 @@ int load_crashdump_segments(struct kexec_info *info, char *mod_cmdline)<br>
+        * 1MB) so that available memory passed in kernel command line will be<br>
+        * aligned to 1MB. This is because kernel create_mapping() wants memory<br>
+        * regions to be aligned to SECTION_SIZE.<br>
++         * The SECTION_SIZE of LPAE kernel is &#39;1UL &lt;&lt; 21&#39; defined in pgtable-3level.h<br>
+        */<br>
+-      elfcorehdr = add_buffer_phys_virt(info, buf, bufsz, bufsz, 1 &lt;&lt; 20,<br>
++      elfcorehdr = add_buffer_phys_virt(info, buf, bufsz, bufsz, 1 &lt;&lt; align_bit_shift,<br>
+                                         crash_reserved_mem.start,<br>
+                                         crash_reserved_mem.end, -1, 0);<br>
+<br>
+--<br>
+1.9.1<br>
+<br>
diff --git a/meta/recipes-kernel/kexec/<a href="http://kexec-tools_2.0.12.bb" target="_blank">ke<wbr>xec-tools_2.0.12.bb</a> b/meta/recipes-kernel/kexec/<a href="http://kexec-tools_2.0.12.bb" target="_blank">ke<wbr>xec-tools_2.0.12.bb</a><br>
index 59376c8..3e38923 100644<br>
--- a/meta/recipes-kernel/kexec/<a href="http://kexec-tools_2.0.12.bb" target="_blank">ke<wbr>xec-tools_2.0.12.bb</a><br>
+++ b/meta/recipes-kernel/kexec/<a href="http://kexec-tools_2.0.12.bb" target="_blank">ke<wbr>xec-tools_2.0.12.bb</a><br>
@@ -8,6 +8,7 @@ SRC_URI += &quot; \<br>
             <a>file://0002-powerpc-change-<wbr>the-memory-size-limit.patch</a> \<br>
             <a>file://0001-purgatory-Pass-r-<wbr>directly-to-linker.patch</a> \<br>
             <a>file://0001-vmcore-dmesg-<wbr>Define-_GNU_SOURCE.patch</a> \<br>
+            <a>
file://kexec-ARM-fix-add_<wbr>buffer_phys_virt-align-issue.<wbr>patch</a> \<br>
          &quot;<br>
<br>
 SRC_URI[md5sum] = &quot;<wbr>10ddaae0e86af54407b164a1f5a39c<wbr>c3&quot;<br>
<span class="m_-918250190295239798hoenzb"><span style="color:#888888">--</span></span><span style="color:#888888"><br>
<span class="m_-918250190295239798hoenzb">1.9.1</span><br>
<br>
<span class="m_-918250190295239798hoenzb">--</span><br>
<span class="m_-918250190295239798hoenzb">______________________________<wbr>_________________</span><br>
<span class="m_-918250190295239798hoenzb">Openembedded-core mailing list</span><br>
<span class="m_-918250190295239798hoenzb"><a href="mailto:Openembedded-core@lists.openembedded.org" target="_blank">Openembedded-core@lists.<wbr>openembedded.org</a></span><br>
<span class="m_-918250190295239798hoenzb"><a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" target="_blank">http://lists.openembedded.org/<wbr>mailman/listinfo/openembedded-<wbr>core</a></span></span><u></u><u></u></p>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div></div></div>
</div>

</blockquote></div><br></div>