Re: [PATCH net-next 3/5] bpftool: implement cgattach command

From: David Ahern
Date: Thu Nov 30 2017 - 11:17:25 EST


On 11/30/17 6:43 AM, Roman Gushchin wrote:
> @@ -75,12 +80,13 @@ static int do_help(int argc, char **argv)
> fprintf(stderr,
> "Usage: %s [OPTIONS] OBJECT { COMMAND | help }\n"
> " %s batch file FILE\n"
> + " %s cgattach FILE CGROUP TYPE\n"

Can you change the order to:
+ " %s cgattach CGROUP TYPE FILE\n"

Makes for better consistency with the detach command in the next patch:
+ " %s cgdetach CGROUP TYPE ID\n"