Re: [PATCH v5 2/3] of/selftest: Add self tests for manipulation of properties

From: Grant Likely
Date: Sun Nov 17 2013 - 17:14:53 EST


On Sat, 16 Nov 2013 19:50:37 +0200, Pantelis Antoniou <panto@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
> Hi Grant,
>
> On Nov 15, 2013, at 7:46 PM, Grant Likely wrote:
>
> > Adds a few simple test cases to ensure that addition, update and removal
> > of device tree node properties works correctly.
> >
> > Signed-off-by: Grant Likely <grant.likely@xxxxxxxxxx>
> > Cc: Rob Herring <rob.herring@xxxxxxxxxxx>
> > Cc: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
> > Cc: David S. Miller <davem@xxxxxxxxxxxxx>
> > Cc: Nathan Fontenot <nfont@xxxxxxxxxxxxxxxxxx>
> > Cc: Pantelis Antoniou <panto@xxxxxxxxxxxxxxxxxxxxxxx>
> > ---
> > drivers/of/selftest.c | 62 +++++++++++++++++++++++++++++++++++++++++++++++++++
> > 1 file changed, 62 insertions(+)
> >
> > diff --git a/drivers/of/selftest.c b/drivers/of/selftest.c
> > index e21012bde639..cb8d3e722f76 100644
> > --- a/drivers/of/selftest.c
> > +++ b/drivers/of/selftest.c
> > @@ -30,6 +30,67 @@ static struct selftest_results {
> > } \
> > }
> >
> > +static void __init of_selftest_dynamic(void)
> > +{
> > + struct device_node *np;
> > + struct property *prop;
> > +
> > + np = of_find_node_by_path("/testcase-data");
> > + if (!np) {
> > + pr_err("missing testcase data\n");
> > + return;
> > + }
> > +
> > + /* Array of 4 properties for the purpose of testing */
> > + prop = kzalloc(sizeof(*prop) * 4, GFP_KERNEL);
> > + if (!prop) {
> > + selftest(0, "kzalloc() failed\n");
> > + return;
> > + }
> > +
> > + /* Add a new property - should pass*/
> > + prop->name = "new-property";
> > + prop->value = "new-property-data";
> > + prop->length = strlen(prop->value);
> > + selftest(of_add_property(np, prop) == 0, "Adding a new property failed\n");
> > +
> > + /* Try to add an existing property - should fail */
> > + prop++;
> > + prop->name = "new-property";
> > + prop->value = "new-property-data-should-fail";
> > + prop->length = strlen(prop->value);
> > + selftest(of_add_property(np, prop) != 0,
> > + "Adding an existing property should have failed\n");
> > +
> > + /* Try to modify an existing property - should pass */
> > + prop->value = "modify-property-data-should-pass";
> > + prop->length = strlen(prop->value);
> > + selftest(of_update_property(np, prop) == 0,
> > + "Updating an existing property should have passed\n");
> > +
> > + /* Try to modify non-existent property - should pass*/
> > + prop++;
> > + prop->name = "modify-property";
> > + prop->value = "modify-missing-property-data-should-pass";
> > + prop->length = strlen(prop->value);
> > + selftest(of_update_property(np, prop) == 0,
> > + "Updating a missing property should have passed\n");
> > +
> > + /* Remove property - should pass */
> > + selftest(of_remove_property(np, prop) == 0,
> > + "Removing a property should have passed\n");
> > +
> > + /* Adding very large property - should pass */
> > + prop++;
> > + prop->name = "large-property-PAGE_SIZEx8";
> > + prop->length = PAGE_SIZE * 8;
> > + prop->value = kzalloc(prop->length, GFP_KERNEL);
> > + selftest(prop->value != NULL, "Unable to allocate large buffer\n");
> > + if (prop->value)
> > + selftest(of_add_property(np, prop) == 0,
> > + "Adding a large property should have passed\n");
> > +}
> > +
> > static void __init of_selftest_parse_phandle_with_args(void)
> > {
> > struct device_node *np;
> > @@ -312,6 +373,7 @@ static int __init of_selftest(void)
> > of_node_put(np);
> >
> > pr_info("start of selftest - you will see error messages\n");
> > + of_selftest_dynamic();
> > of_selftest_parse_phandle_with_args();
> > of_selftest_property_match_string();
> > of_selftest_parse_interrupts();
> > --
> > 1.8.3.2
> >
>
> I like this. I'll add my overlay test cases somewhere around here.
>
> I might need to introduce some new helper functions in order to generate
> the overlay completely programmatically.

That would be good. I also really want to get the overlay code working
so I can use it for loading the testcase data on any running system.
:-)

g.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/