<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    <div class="moz-cite-prefix">On 06/16/2014 06:56 AM, Martin Jansa
      wrote:<br>
    </div>
    <blockquote cite="mid:20140615225618.GL2428@jama" type="cite">
      <pre wrap="">FYI: this will probably break
meta-initramfs/recipes-kernel/kexec/kexec-tools-klibc_2.0.2.bb

because it doesn't have kexec-tools directory in FILESPATH

WARNING: Unable to get checksum for kexec-tools-klibc SRC_URI entry
kexec-tools-Refine-kdump-device_tree-sort.patch: file could not be found</pre>
    </blockquote>
    OK, I will send a new patch to fix this issue.<br>
    <br>
    Best Regards<br>
    Chong<br>
    <blockquote cite="mid:20140615225618.GL2428@jama" type="cite">
      <pre wrap="">

On Fri, Jun 13, 2014 at 11:32:18AM +0000, <a class="moz-txt-link-abbreviated" href="mailto:git@git.openembedded.org">git@git.openembedded.org</a> wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">Module: openembedded-core.git
Branch: master
Commit: c8722b510f779cd20757477a7f7a7a2a35b9a9c5
URL:    <a class="moz-txt-link-freetext" href="http://git.openembedded.org/?p=openembedded-core.git&amp;a=commit;h=c8722b510f779cd20757477a7f7a7a2a35b9a9c5">http://git.openembedded.org/?p=openembedded-core.git&amp;a=commit;h=c8722b510f779cd20757477a7f7a7a2a35b9a9c5</a>

Author: Chong Lu <a class="moz-txt-link-rfc2396E" href="mailto:Chong.Lu@windriver.com">&lt;Chong.Lu@windriver.com&gt;</a>
Date:   Tue Jun 10 18:13:39 2014 +0800

kexec-tools: Refine kdump device_tree sort

The commit b02d735bf was to rearrange the device-tree entries, and
assumed that these entries are sorted in the ascending order. but
acctually when I was validating kexec and kdump, the order of
serial node still is changed. So the patch is to sort these entries
by the directory name in ascending order.

Signed-off-by: Yang Wei <a class="moz-txt-link-rfc2396E" href="mailto:Wei.Yang@windriver.com">&lt;Wei.Yang@windriver.com&gt;</a>
Signed-off-by: Chong Lu <a class="moz-txt-link-rfc2396E" href="mailto:Chong.Lu@windriver.com">&lt;Chong.Lu@windriver.com&gt;</a>
Signed-off-by: Saul Wold <a class="moz-txt-link-rfc2396E" href="mailto:sgw@linux.intel.com">&lt;sgw@linux.intel.com&gt;</a>
Signed-off-by: Richard Purdie <a class="moz-txt-link-rfc2396E" href="mailto:richard.purdie@linuxfoundation.org">&lt;richard.purdie@linuxfoundation.org&gt;</a>

---

 meta/recipes-kernel/kexec/kexec-tools.inc          |  3 +-
 ...kexec-tools-Refine-kdump-device_tree-sort.patch | 82 ++++++++++++++++++++++
 2 files changed, 84 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-kernel/kexec/kexec-tools.inc b/meta/recipes-kernel/kexec/kexec-tools.inc
index 50b448c..6a690c6 100644
--- a/meta/recipes-kernel/kexec/kexec-tools.inc
+++ b/meta/recipes-kernel/kexec/kexec-tools.inc
@@ -8,7 +8,8 @@ LIC_FILES_CHKSUM = <a class="moz-txt-link-rfc2396E" href="file://COPYING;md5=ea5bed2f60d357618ca161ad539f7c0a\file://kexec/kexec.c;beginline=1;endline=20;md5=af10f6ae4a8715965e648aa687ad3e09">"file://COPYING;md5=ea5bed2f60d357618ca161ad539f7c0a \
                     file://kexec/kexec.c;beginline=1;endline=20;md5=af10f6ae4a8715965e648aa687ad3e09"</a>
 DEPENDS = "zlib xz"
 
-SRC_URI = "${KERNELORG_MIRROR}/linux/utils/kernel/kexec/kexec-tools-${PV}.tar.gz"
+SRC_URI = "${KERNELORG_MIRROR}/linux/utils/kernel/kexec/kexec-tools-${PV}.tar.gz \
+           <a class="moz-txt-link-freetext" href="file://kexec-tools-Refine-kdump-device_tree-sort.patch">file://kexec-tools-Refine-kdump-device_tree-sort.patch</a>"
 
 inherit autotools-brokensep
 
diff --git a/meta/recipes-kernel/kexec/kexec-tools/kexec-tools-Refine-kdump-device_tree-sort.patch b/meta/recipes-kernel/kexec/kexec-tools/kexec-tools-Refine-kdump-device_tree-sort.patch
new file mode 100644
index 0000000..ab1fe96
--- /dev/null
+++ b/meta/recipes-kernel/kexec/kexec-tools/kexec-tools-Refine-kdump-device_tree-sort.patch
@@ -0,0 +1,82 @@
+kexec-tools: Refine kdump device_tree sort
+
+The commit b02d735bf was to rearrange the device-tree entries, and
+assumed that these entries are sorted in the ascending order. but
+acctually when I was validating kexec and kdump, the order of
+serial node still is changed. So the patch is to sort these entries
+by the directory name in ascending order.
+
+Upstream-Status: Pending
+
+Signed-off-by: Yang Wei <a class="moz-txt-link-rfc2396E" href="mailto:Wei.Yang@windriver.com">&lt;Wei.Yang@windriver.com&gt;</a>
+Signed-off-by: Chong Lu <a class="moz-txt-link-rfc2396E" href="mailto:Chong.Lu@windriver.com">&lt;Chong.Lu@windriver.com&gt;</a>
+---
+ kexec/arch/ppc/fs2dt.c |   13 ++++++++++---
+ kexec/fs2dt.c          |   13 ++++++++++---
+ 2 files changed, 20 insertions(+), 6 deletions(-)
+
+diff --git a/kexec/arch/ppc/fs2dt.c b/kexec/arch/ppc/fs2dt.c
+index 4121c7d..cc04b04 100644
+--- a/kexec/arch/ppc/fs2dt.c
++++ b/kexec/arch/ppc/fs2dt.c
+@@ -296,6 +296,9 @@ static int comparefunc(const void *dentry1, const void *dentry2)
+ {
+         char *str1 = (*(struct dirent **)dentry1)-&gt;d_name;
+         char *str2 = (*(struct dirent **)dentry2)-&gt;d_name;
++        char* ptr1 = strchr(str1, '@');
++        char* ptr2 = strchr(str2, '@');
++        int len1, len2;
+ 
+         /*
+          * strcmp scans from left to right and fails to idetify for some
+@@ -303,9 +306,13 @@ static int comparefunc(const void *dentry1, const void *dentry2)
+          * Therefore, we get the wrong sorted order like memory@10000000 and
+          * memory@f000000.
+          */
+-        if (strchr(str1, '@') &amp;&amp; strchr(str2, '@') &amp;&amp;
+-                (strlen(str1) &gt; strlen(str2)))
+-                return 1;
++        if (ptr1 &amp;&amp; ptr2) {
++                len1 = ptr1 - str1;
++                len2 = ptr2 - str2;
++                if (!strncmp(str1, str2, len1 &gt;len2 ? len1: len2) &amp;&amp;
++                                        (strlen(str1) &gt; strlen(str2)))
++                                return 1;
++        }
+ 
+         return strcmp(str1, str2);
+ }
+diff --git a/kexec/fs2dt.c b/kexec/fs2dt.c
+index 5e6b98d..1c0345b 100644
+--- a/kexec/fs2dt.c
++++ b/kexec/fs2dt.c
+@@ -475,6 +475,9 @@ static int comparefunc(const struct dirent **dentry1,
+ {
+         char *str1 = (*(struct dirent **)dentry1)-&gt;d_name;
+         char *str2 = (*(struct dirent **)dentry2)-&gt;d_name;
++        char* ptr1 = strchr(str1, '@');
++        char* ptr2 = strchr(str2, '@');
++        int len1, len2;
+ 
+         /*
+          * strcmp scans from left to right and fails to idetify for some
+@@ -482,9 +485,13 @@ static int comparefunc(const struct dirent **dentry1,
+          * Therefore, we get the wrong sorted order like memory@10000000 and
+          * memory@f000000.
+          */
+-        if (strchr(str1, '@') &amp;&amp; strchr(str2, '@') &amp;&amp;
+-                (strlen(str1) &gt; strlen(str2)))
+-                return 1;
++        if (ptr1 &amp;&amp; ptr2) {
++                len1 = ptr1 - str1;
++                len2 = ptr2 - str2;
++                if (!strncmp(str1, str2, len1 &gt;len2 ? len1: len2) &amp;&amp;
++                                        (strlen(str1) &gt; strlen(str2)))
++                                return 1;
++        }
+ 
+         return strcmp(str1, str2);
+ }
+-- 
+1.7.9.5
+

-- 
_______________________________________________
Openembedded-commits mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Openembedded-commits@lists.openembedded.org">Openembedded-commits@lists.openembedded.org</a>
<a class="moz-txt-link-freetext" href="http://lists.openembedded.org/mailman/listinfo/openembedded-commits">http://lists.openembedded.org/mailman/listinfo/openembedded-commits</a>
</pre>
      </blockquote>
      <pre wrap="">
</pre>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
    </blockquote>
    <br>
  </body>
</html>