From: Rob Herring <robherring2@gmail.com> To: Michal Simek <monstr@monstr.eu> Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Grant Likely <grant.likely@linaro.org>, Benjamin Herrenschmidt <benh@kernel.crashing.org>, Paul Mackerras <paulus@samba.org>, linuxppc-dev <linuxppc-dev@lists.ozlabs.org> Subject: Re: [PATCH 09/20] of/fdt: create common debugfs Date: Fri, 4 Apr 2014 08:32:41 -0500 [thread overview] Message-ID: <CAL_Jsq+tf4wgjB7uTNipWEj5d6tPkd6QwO_xiW87P51Y_y48vw@mail.gmail.com> (raw) In-Reply-To: <533EB207.1020203@monstr.eu> On Fri, Apr 4, 2014 at 8:22 AM, Michal Simek <monstr@monstr.eu> wrote: > On 04/04/2014 03:00 PM, Rob Herring wrote: >> On Fri, Apr 4, 2014 at 7:16 AM, Michal Simek <monstr@monstr.eu> wrote: >>> On 04/04/2014 12:16 AM, Rob Herring wrote: >>>> From: Rob Herring <robh@kernel.org> >>>> >>>> Both powerpc and microblaze have the same FDT blob in debugfs feature. >>>> Move this to common location and remove the powerpc and microblaze >>>> implementations. This feature could become more useful when FDT >>>> overlay support is added. >> >> [snip] > Anyway I am testing it for microblaze and getting problem > caused by this patch: > commit 3d2ee8571ac0580d49c3f41fa28336289934900a > Author: Rob Herring <robh@kernel.org> > Date: Wed Apr 2 15:10:14 2014 -0500 > > of/fdt: Convert FDT functions to use libfdt > > And reason is that in unflatten_dt_node() > > pathp = fdt_get_name(blob, *poffset, &l); > > is returning NULL > and here > /* version 0x10 has a more compact unit name here instead of the full > * path. we accumulate the full path size using "fpsize", we'll rebuild > * it later. We detect this because the first character of the name is > * not '/'. > */ > if ((*pathp) != '/') { > > code is trying to read it which is causing this kernel bug: > Oops: kernel access of bad area, sig: 11 > > It means fdt_next_node(is doing something wrong) > > Any easy way how to debug it? I didn't think fdt_get_path should fail. Can you add a print of *poffset and pathp values. Rob
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robherring2@gmail.com> To: Michal Simek <monstr@monstr.eu> Cc: Grant Likely <grant.likely@linaro.org>, linuxppc-dev <linuxppc-dev@lists.ozlabs.org>, Paul Mackerras <paulus@samba.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org> Subject: Re: [PATCH 09/20] of/fdt: create common debugfs Date: Fri, 4 Apr 2014 08:32:41 -0500 [thread overview] Message-ID: <CAL_Jsq+tf4wgjB7uTNipWEj5d6tPkd6QwO_xiW87P51Y_y48vw@mail.gmail.com> (raw) In-Reply-To: <533EB207.1020203@monstr.eu> On Fri, Apr 4, 2014 at 8:22 AM, Michal Simek <monstr@monstr.eu> wrote: > On 04/04/2014 03:00 PM, Rob Herring wrote: >> On Fri, Apr 4, 2014 at 7:16 AM, Michal Simek <monstr@monstr.eu> wrote: >>> On 04/04/2014 12:16 AM, Rob Herring wrote: >>>> From: Rob Herring <robh@kernel.org> >>>> >>>> Both powerpc and microblaze have the same FDT blob in debugfs feature. >>>> Move this to common location and remove the powerpc and microblaze >>>> implementations. This feature could become more useful when FDT >>>> overlay support is added. >> >> [snip] > Anyway I am testing it for microblaze and getting problem > caused by this patch: > commit 3d2ee8571ac0580d49c3f41fa28336289934900a > Author: Rob Herring <robh@kernel.org> > Date: Wed Apr 2 15:10:14 2014 -0500 > > of/fdt: Convert FDT functions to use libfdt > > And reason is that in unflatten_dt_node() > > pathp = fdt_get_name(blob, *poffset, &l); > > is returning NULL > and here > /* version 0x10 has a more compact unit name here instead of the full > * path. we accumulate the full path size using "fpsize", we'll rebuild > * it later. We detect this because the first character of the name is > * not '/'. > */ > if ((*pathp) != '/') { > > code is trying to read it which is causing this kernel bug: > Oops: kernel access of bad area, sig: 11 > > It means fdt_next_node(is doing something wrong) > > Any easy way how to debug it? I didn't think fdt_get_path should fail. Can you add a print of *poffset and pathp values. Rob
next prev parent reply other threads:[~2014-04-04 13:32 UTC|newest] Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-04-03 22:16 [PATCH 00/20] FDT clean-ups and libfdt support Rob Herring 2014-04-03 22:16 ` Rob Herring 2014-04-03 22:16 ` Rob Herring 2014-04-03 22:16 ` Rob Herring 2014-04-03 22:16 ` Rob Herring 2014-04-03 22:16 ` [PATCH 01/20] mips: octeon: convert to use unflatten_and_copy_device_tree Rob Herring 2014-04-07 17:46 ` Andreas Herrmann 2014-04-16 1:02 ` Rob Herring 2014-04-03 22:16 ` [PATCH 02/20] mips: lantiq: copy built-in DTB out of init section Rob Herring 2014-04-03 22:16 ` [PATCH 03/20] mips: xlp: " Rob Herring 2014-04-03 22:16 ` [PATCH 04/20] mips: ralink: convert to use unflatten_and_copy_device_tree Rob Herring 2014-04-03 22:16 ` [PATCH 05/20] ARM: dt: use default early_init_dt_alloc_memory_arch Rob Herring 2014-04-03 22:16 ` Rob Herring 2014-04-03 22:16 ` [PATCH 06/20] c6x: convert fdt pointers to opaque pointers Rob Herring 2014-04-04 7:57 ` Geert Uytterhoeven 2015-03-07 9:16 ` Nishanth Menon 2014-04-03 22:16 ` [PATCH 07/20] mips: " Rob Herring 2014-04-03 22:16 ` [PATCH 08/20] of/fdt: consolidate built-in dtb section variables Rob Herring 2014-04-03 22:16 ` Rob Herring 2014-04-03 22:16 ` Rob Herring 2014-04-07 10:54 ` James Hogan 2014-04-07 10:54 ` James Hogan 2014-04-07 10:54 ` James Hogan 2014-04-08 4:23 ` Vineet Gupta 2014-04-08 4:23 ` Vineet Gupta 2014-04-08 4:23 ` Vineet Gupta 2014-04-03 22:16 ` [PATCH 09/20] of/fdt: create common debugfs Rob Herring 2014-04-03 22:16 ` Rob Herring 2014-04-04 12:16 ` Michal Simek 2014-04-04 12:16 ` Michal Simek 2014-04-04 12:22 ` Michal Simek 2014-04-04 12:22 ` Michal Simek 2014-04-04 13:00 ` Rob Herring 2014-04-04 13:00 ` Rob Herring 2014-04-04 13:22 ` Michal Simek 2014-04-04 13:22 ` Michal Simek 2014-04-04 13:32 ` Rob Herring [this message] 2014-04-04 13:32 ` Rob Herring 2014-04-04 14:11 ` Michal Simek 2014-04-04 14:11 ` Michal Simek 2014-04-07 0:42 ` Rob Herring 2014-04-07 0:42 ` Rob Herring 2014-04-07 7:04 ` Michal Simek 2014-04-07 7:04 ` Michal Simek 2014-04-03 22:16 ` [PATCH 10/20] of/fdt: remove some unneeded includes Rob Herring 2014-04-03 22:16 ` [PATCH 11/20] of/fdt: remove unused of_scan_flat_dt_by_path Rob Herring 2014-04-03 22:16 ` [PATCH 12/20] of/fdt: update of_get_flat_dt_prop in prep for libfdt Rob Herring 2014-04-03 22:16 ` [PATCH 13/20] of/fdt: Convert FDT functions to use libfdt Rob Herring 2014-04-03 22:16 ` [PATCH 14/20] of/fdt: use libfdt accessors for header data Rob Herring 2014-04-08 3:54 ` Max Filippov 2014-04-03 22:16 ` [PATCH 15/20] of/fdt: move memreserve and dtb memory reservations into core Rob Herring 2014-04-03 22:16 ` [PATCH 16/20] build: add libfdt include path globally Rob Herring 2014-04-03 22:17 ` [PATCH 17/20] powerpc: use libfdt accessors for header data Rob Herring 2014-04-03 22:17 ` [PATCH 18/20] x86: " Rob Herring 2014-04-03 22:17 ` [PATCH 19/20] of/fdt: convert initial_boot_params to opaque pointer Rob Herring 2014-04-03 22:17 ` [PATCH 20/20] of: push struct boot_param_header and defines into powerpc Rob Herring 2014-04-03 22:17 ` Rob Herring
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAL_Jsq+tf4wgjB7uTNipWEj5d6tPkd6QwO_xiW87P51Y_y48vw@mail.gmail.com \ --to=robherring2@gmail.com \ --cc=benh@kernel.crashing.org \ --cc=grant.likely@linaro.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=monstr@monstr.eu \ --cc=paulus@samba.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.