Re: [PATCH v2 07/13] of: unittest: Cleanup partially-applied overlays

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rob,

On Tue, Aug 1, 2023 at 10:50 PM Rob Herring <robh+dt@xxxxxxxxxx> wrote:
> On Fri, Jul 28, 2023 at 2:50 AM Geert Uytterhoeven
> <geert+renesas@xxxxxxxxx> wrote:
> > When of_overlay_fdt_apply() fails, the changeset may be partially
> > applied, and the caller is still expected to call of_overlay_remove() to
> > clean up this partial state.  However, overlay_17 is the only test that
> > takes care of cleaning up after an (expected) failure.
> >
> > Instead of adding cleanup code to each individual test, extend
> > overlay_info with the optional expected return value of
> > of_overlay_remove(), and handle cleanup in the overlay_data_apply()
> > helper.  While at it, simplify the end marker in the overlay_info table.
> >
> > Update the expected error output for errors during the newly cleanup.
> >
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> > ---
> > v2:
> >   - No changes.

> > --- a/drivers/of/unittest.c
> > +++ b/drivers/of/unittest.c

> > @@ -3689,9 +3715,19 @@ static __init void of_unittest_overlay_high_level(void)
> >         unittest(overlay_data_apply("overlay_bad_phandle", NULL),
> >                  "Adding overlay 'overlay_bad_phandle' failed\n");
> >
> > +       EXPECT_BEGIN(KERN_ERR,
> > +                    "OF: changeset: remove_property failed @/testcase-data-2/substation@100/hvac-medium-2/name");
> > +       EXPECT_BEGIN(KERN_ERR,
> > +                    "OF: resolver: overlay phandle fixup failed: -22");
> > +
> >         unittest(overlay_data_apply("overlay_bad_symbol", NULL),
> >                  "Adding overlay 'overlay_bad_symbol' failed\n");
> >
> > +       EXPECT_END(KERN_ERR,
> > +                  "OF: resolver: overlay phandle fixup failed: -22");
>
> I'm seeing "OF: Error reverting changeset (-19)" here instead.
> Cut-n-paste error?

Indeed. Thanks for pointing this out!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

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



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux