LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* kexec fails if OF_UNITTEST=y (was: Re: [PATCH v2] Removes OF_UNITTEST dependency on OF_DYNAMIC config symbol)
@ 2015-03-16 10:58 Geert Uytterhoeven
  2015-03-17  0:25 ` Rob Herring
  0 siblings, 1 reply; 3+ messages in thread
From: Geert Uytterhoeven @ 2015-03-16 10:58 UTC (permalink / raw)
  To: Gaurav Minocha; +Cc: devicetree, Grant Likely, Rob Herring, kexec, linux-kernel

On Fri, Jan 23, 2015 at 2:03 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> On Sun, Jan 11, 2015 at 8:19 AM, Gaurav Minocha
> <gaurav.minocha.os@gmail.com> wrote:
>> This patch intends to remove the unittests dependency on
>> the functions defined in dynamic.c. So, rather than calling
>> of_attach_node defined in dynamic.c, minimal functionality
>> required to attach a new node is re-defined in unittest.c.
>> Also, now after executing the tests the test data is not
>> removed from the device tree so there is no need to call
>> of_detach_node.
>
> Could there be unwanted side effects of not removing the test data?

Unfortunately I found a regression introduced by commit 3ce04b4a9fdc30b6
("Removes OF_UNITTEST dependency on OF_DYNAMIC config symbol").

If the test data is still present, kexec (kexec-tools 2.0.7 released 24
November 2014, 1:2.0.7-5 in Debian) fails with:

unrecoverable error: short read
from"/proc/device-tree//testcase-data/large-property-PAGE_SIZEx8"

Granted, this is a bug in kexec-tools, but I'm reporting it anyway, as it is
a kernel regression with existing userspace.

I believe this bug was fixed in kexec-tools by commit d1932cd592e2a6aa
("kexec/fs2dt: Use slurp_file_len to avoid partial read of files"), but I
haven't tested the fix yet.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: kexec fails if OF_UNITTEST=y (was: Re: [PATCH v2] Removes OF_UNITTEST dependency on OF_DYNAMIC config symbol)
  2015-03-16 10:58 kexec fails if OF_UNITTEST=y (was: Re: [PATCH v2] Removes OF_UNITTEST dependency on OF_DYNAMIC config symbol) Geert Uytterhoeven
@ 2015-03-17  0:25 ` Rob Herring
  2015-03-17  6:52   ` Geert Uytterhoeven
  0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring @ 2015-03-17  0:25 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Gaurav Minocha, devicetree, Grant Likely, Rob Herring, kexec,
	linux-kernel

On Mon, Mar 16, 2015 at 5:58 AM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> On Fri, Jan 23, 2015 at 2:03 PM, Geert Uytterhoeven
> <geert@linux-m68k.org> wrote:
>> On Sun, Jan 11, 2015 at 8:19 AM, Gaurav Minocha
>> <gaurav.minocha.os@gmail.com> wrote:
>>> This patch intends to remove the unittests dependency on
>>> the functions defined in dynamic.c. So, rather than calling
>>> of_attach_node defined in dynamic.c, minimal functionality
>>> required to attach a new node is re-defined in unittest.c.
>>> Also, now after executing the tests the test data is not
>>> removed from the device tree so there is no need to call
>>> of_detach_node.
>>
>> Could there be unwanted side effects of not removing the test data?
>
> Unfortunately I found a regression introduced by commit 3ce04b4a9fdc30b6
> ("Removes OF_UNITTEST dependency on OF_DYNAMIC config symbol").
>
> If the test data is still present, kexec (kexec-tools 2.0.7 released 24
> November 2014, 1:2.0.7-5 in Debian) fails with:
>
> unrecoverable error: short read
> from"/proc/device-tree//testcase-data/large-property-PAGE_SIZEx8"
>
> Granted, this is a bug in kexec-tools, but I'm reporting it anyway, as it is
> a kernel regression with existing userspace.
>
> I believe this bug was fixed in kexec-tools by commit d1932cd592e2a6aa
> ("kexec/fs2dt: Use slurp_file_len to avoid partial read of files"), but I
> haven't tested the fix yet.

Is OF_UNITTEST enabled by default in Debian or this is a custom
kernel? Perhaps we should require a command line option to actually
populate the test data and run the tests?

We just need more tests to slow down the boot enough people will not
enable them by default. ;)

Rob

>
> Gr{oetje,eeting}s,
>
>                         Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: kexec fails if OF_UNITTEST=y (was: Re: [PATCH v2] Removes OF_UNITTEST dependency on OF_DYNAMIC config symbol)
  2015-03-17  0:25 ` Rob Herring
@ 2015-03-17  6:52   ` Geert Uytterhoeven
  0 siblings, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2015-03-17  6:52 UTC (permalink / raw)
  To: Rob Herring
  Cc: Gaurav Minocha, devicetree, Grant Likely, Rob Herring, kexec,
	linux-kernel

Hi Rob,

On Tue, Mar 17, 2015 at 1:25 AM, Rob Herring <robherring2@gmail.com> wrote:
> On Mon, Mar 16, 2015 at 5:58 AM, Geert Uytterhoeven
> <geert@linux-m68k.org> wrote:
>> On Fri, Jan 23, 2015 at 2:03 PM, Geert Uytterhoeven
>> <geert@linux-m68k.org> wrote:
>>> On Sun, Jan 11, 2015 at 8:19 AM, Gaurav Minocha
>>> <gaurav.minocha.os@gmail.com> wrote:
>>>> This patch intends to remove the unittests dependency on
>>>> the functions defined in dynamic.c. So, rather than calling
>>>> of_attach_node defined in dynamic.c, minimal functionality
>>>> required to attach a new node is re-defined in unittest.c.
>>>> Also, now after executing the tests the test data is not
>>>> removed from the device tree so there is no need to call
>>>> of_detach_node.
>>>
>>> Could there be unwanted side effects of not removing the test data?
>>
>> Unfortunately I found a regression introduced by commit 3ce04b4a9fdc30b6
>> ("Removes OF_UNITTEST dependency on OF_DYNAMIC config symbol").
>>
>> If the test data is still present, kexec (kexec-tools 2.0.7 released 24
>> November 2014, 1:2.0.7-5 in Debian) fails with:
>>
>> unrecoverable error: short read
>> from"/proc/device-tree//testcase-data/large-property-PAGE_SIZEx8"
>>
>> Granted, this is a bug in kexec-tools, but I'm reporting it anyway, as it is
>> a kernel regression with existing userspace.
>>
>> I believe this bug was fixed in kexec-tools by commit d1932cd592e2a6aa
>> ("kexec/fs2dt: Use slurp_file_len to avoid partial read of files"), but I
>> haven't tested the fix yet.
>
> Is OF_UNITTEST enabled by default in Debian or this is a custom
> kernel? Perhaps we should require a command line option to actually
> populate the test data and run the tests?

It's not that bad, this is a custom kernel. I just had it still
enabled by accident.

I guess making it modular is not that easy?

> We just need more tests to slow down the boot enough people will not
> enable them by default. ;)

Apart from slowing down the boot, most tests don't have side effects.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-03-17  6:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-16 10:58 kexec fails if OF_UNITTEST=y (was: Re: [PATCH v2] Removes OF_UNITTEST dependency on OF_DYNAMIC config symbol) Geert Uytterhoeven
2015-03-17  0:25 ` Rob Herring
2015-03-17  6:52   ` Geert Uytterhoeven

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).