Subject: Re: [IPROUTE2 PATCH][GENETLINK] Add controller support for new features exposed

From: jamal <[email-masked]>
Date: Mon, 11 Dec 2006 20:31:04 -0500

On Mon, 2006-11-12 at 17:37 +0100, Ingo Oeser wrote:
> Hi Jamal,
> 
> jamal wrote:
> > +
> > +       if (fl & GENL_ADMIN_PERM)
> > +               fprintf(fp, " requires admin permission;");
> > +       if (fl & GENL_CMD_CAP_DO)
> > +               fprintf(fp, " can doit;");
> > +       if (fl & GENL_CMD_CAP_DUMP)
> > +               fprintf(fp, " can dumpit;");
> > +       if (fl & GENL_CMD_CAP_HASPOL)
> > +               fprintf(fp, " has policy");
> 
> Can't you think of sth. more admin/user friendly instead of "doit" and "dumpit"?
> iproute is cryptic enough already :-)
> 

I have to think a little about the best way to make this more friendly. 
If you have any suggestion please shoot. Lets have Stephen merge all the
patches then we can revisit. If you have time take a look at the genl
howto that Paul Moore has been tossing around and see if you can pick
some better language.

cheers,
jamal