Hi Bjorn,
Il giorno sab 13 mar 2021 alle ore 01:02 Bjorn Andersson
<bjorn.anders...@linaro.org> ha scritto:
>
> Parse and pass IFLA_RMNET_FLAGS to the kernel, to allow changing the
> flags from the default of ingress-aggregate only.
>
> Signed-off-by: Bjorn Andersson <bjorn.anders...@linaro.org>
> ---
> ip/iplink_rmnet.c | 42 ++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 42 insertions(+)
>
> diff --git a/ip/iplink_rmnet.c b/ip/iplink_rmnet.c
> index 1d16440c6900..8a488f3d0316 100644
> --- a/ip/iplink_rmnet.c
> +++ b/ip/iplink_rmnet.c
> @@ -16,6 +16,10 @@ static void print_explain(FILE *f)
> {
> fprintf(f,
> "Usage: ... rmnet mux_id MUXID\n"
> + " [ingress-deaggregation]\n"
> + " [ingress-commands]\n"
> + " [ingress-chksumv4]\n"
> + " [egress-chksumv4]\n"
> "\n"
> "MUXID := 1-254\n"
> );
> @@ -29,6 +33,7 @@ static void explain(void)
> static int rmnet_parse_opt(struct link_util *lu, int argc, char **argv,
> struct nlmsghdr *n)
> {
> + struct ifla_rmnet_flags flags = { };
> __u16 mux_id;
>
> while (argc > 0) {
> @@ -37,6 +42,18 @@ static int rmnet_parse_opt(struct link_util *lu, int argc,
> char **argv,
> if (get_u16(&mux_id, *argv, 0))
> invarg("mux_id is invalid", *argv);
> addattr16(n, 1024, IFLA_RMNET_MUX_ID, mux_id);
> + } else if (matches(*argv, "ingress-deaggregation") == 0) {
> + flags.mask = ~0;
> + flags.flags |= RMNET_FLAGS_INGRESS_DEAGGREGATION;
> + } else if (matches(*argv, "ingress-commands") == 0) {
> + flags.mask = ~0;
> + flags.flags |= RMNET_FLAGS_INGRESS_MAP_COMMANDS;
> + } else if (matches(*argv, "ingress-chksumv4") == 0) {
> + flags.mask = ~0;
> + flags.flags |= RMNET_FLAGS_INGRESS_MAP_CKSUMV4;
> + } else if (matches(*argv, "egress-chksumv4") == 0) {
> + flags.mask = ~0;
> + flags.flags |= RMNET_FLAGS_EGRESS_MAP_CKSUMV4;
> } else if (matches(*argv, "help") == 0) {
> explain();
> return -1;
> @@ -48,11 +65,28 @@ static int rmnet_parse_opt(struct link_util *lu, int
> argc, char **argv,
> argc--, argv++;
> }
>
> + if (flags.mask)
> + addattr_l(n, 1024, IFLA_RMNET_FLAGS, &flags, sizeof(flags));
> +
> return 0;
> }
>
> +static void rmnet_print_flags(FILE *fp, __u32 flags)
> +{
> + if (flags & RMNET_FLAGS_INGRESS_DEAGGREGATION)
> + print_string(PRINT_ANY, NULL, "%s ", "ingress-deaggregation");
> + if (flags & RMNET_FLAGS_INGRESS_MAP_COMMANDS)
> + print_string(PRINT_ANY, NULL, "%s ", "ingress-commands");
> + if (flags & RMNET_FLAGS_INGRESS_MAP_CKSUMV4)
> + print_string(PRINT_ANY, NULL, "%s ", "ingress-chksumv4");
> + if (flags & RMNET_FLAGS_EGRESS_MAP_CKSUMV4)
> + print_string(PRINT_ANY, NULL, "%s ", "egress-cksumv4");
> +}
> +
> static void rmnet_print_opt(struct link_util *lu, FILE *f, struct rtattr
> *tb[])
> {
> + struct ifla_vlan_flags *flags;
just for my understanding, why not struct ifla_rmnet_flags (though
they are exactly the same)?
Thanks,
Daniele
> +
> if (!tb)
> return;
>
> @@ -64,6 +98,14 @@ static void rmnet_print_opt(struct link_util *lu, FILE *f,
> struct rtattr *tb[])
> "mux_id",
> "mux_id %u ",
> rta_getattr_u16(tb[IFLA_RMNET_MUX_ID]));
> +
> + if (tb[IFLA_RMNET_FLAGS]) {
> + if (RTA_PAYLOAD(tb[IFLA_RMNET_FLAGS]) < sizeof(*flags))
> + return;
> + flags = RTA_DATA(tb[IFLA_RMNET_FLAGS]);
> +
> + rmnet_print_flags(f, flags->flags);
> + }
> }
>
> static void rmnet_print_help(struct link_util *lu, int argc, char **argv,
> --
> 2.28.0
>
