Message ID | 1551382150-14662-1-git-send-email-moshele@mellanox.com |
---|---|
State | Accepted |
Delegated to: | Simon Horman |
Headers | show |
Series | [ovs-dev] netdev-tc-offloads: Improve log message for icmpv6 offload not supported | expand |
On 2/28/2019 11:29 AM, Moshe Levi wrote: > Signed-off-by: Moshe Levi <moshele@mellanox.com> > --- > lib/netdev-tc-offloads.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c > index 36bce15..e65f5b9 100644 > --- a/lib/netdev-tc-offloads.c > +++ b/lib/netdev-tc-offloads.c > @@ -1015,12 +1015,12 @@ test_key_and_mask(struct match *match) > key->nw_proto == IPPROTO_ICMPV6) { > if (mask->tp_src) { > VLOG_DBG_RL(&rl, > - "offloading attribute icmp_type isn't supported"); > + "offloading attribute icmpv6_type isn't supported"); > return EOPNOTSUPP; > } > if (mask->tp_dst) { > VLOG_DBG_RL(&rl, > - "offloading attribute icmp_code isn't supported"); > + "offloading attribute icmpv6_code isn't supported"); > return EOPNOTSUPP; > } > } else if (key->dl_type == htons(OFP_DL_TYPE_NOT_ETH_TYPE)) { Nice catch... LGTM Reviewed-by: Greg Rose <gvrose8192@gmail.com>
On 01/03/2019 02:26, Gregory Rose wrote: > > > On 2/28/2019 11:29 AM, Moshe Levi wrote: >> Signed-off-by: Moshe Levi <moshele@mellanox.com> >> --- >> lib/netdev-tc-offloads.c | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c >> index 36bce15..e65f5b9 100644 >> --- a/lib/netdev-tc-offloads.c >> +++ b/lib/netdev-tc-offloads.c >> @@ -1015,12 +1015,12 @@ test_key_and_mask(struct match *match) >> key->nw_proto == IPPROTO_ICMPV6) { >> if (mask->tp_src) { >> VLOG_DBG_RL(&rl, >> - "offloading attribute icmp_type isn't supported"); >> + "offloading attribute icmpv6_type isn't supported"); >> return EOPNOTSUPP; >> } >> if (mask->tp_dst) { >> VLOG_DBG_RL(&rl, >> - "offloading attribute icmp_code isn't supported"); >> + "offloading attribute icmpv6_code isn't supported"); >> return EOPNOTSUPP; >> } >> } else if (key->dl_type == htons(OFP_DL_TYPE_NOT_ETH_TYPE)) { > > Nice catch... > > LGTM > > Reviewed-by: Greg Rose <gvrose8192@gmail.com> > +simon maybe missed it. > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmail.openvswitch.org%2Fmailman%2Flistinfo%2Fovs-dev&data=02%7C01%7Croid%40mellanox.com%7Cdc97de23fe79400d229408d69dddd597%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C636869973437275008&sdata=KVScobN3UPPLb0E3DoLLD5uu%2BOOjM2fyKKhqm8tGf28%3D&reserved=0
On Sun, Mar 17, 2019 at 02:29:45PM +0000, Roi Dayan wrote: > > > On 01/03/2019 02:26, Gregory Rose wrote: > > > > > > On 2/28/2019 11:29 AM, Moshe Levi wrote: > >> Signed-off-by: Moshe Levi <moshele@mellanox.com> > >> --- > >> lib/netdev-tc-offloads.c | 4 ++-- > >> 1 files changed, 2 insertions(+), 2 deletions(-) > >> > >> diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c > >> index 36bce15..e65f5b9 100644 > >> --- a/lib/netdev-tc-offloads.c > >> +++ b/lib/netdev-tc-offloads.c > >> @@ -1015,12 +1015,12 @@ test_key_and_mask(struct match *match) > >> key->nw_proto == IPPROTO_ICMPV6) { > >> if (mask->tp_src) { > >> VLOG_DBG_RL(&rl, > >> - "offloading attribute icmp_type isn't supported"); > >> + "offloading attribute icmpv6_type isn't supported"); > >> return EOPNOTSUPP; > >> } > >> if (mask->tp_dst) { > >> VLOG_DBG_RL(&rl, > >> - "offloading attribute icmp_code isn't supported"); > >> + "offloading attribute icmpv6_code isn't supported"); > >> return EOPNOTSUPP; > >> } > >> } else if (key->dl_type == htons(OFP_DL_TYPE_NOT_ETH_TYPE)) { > > > > Nice catch... > > > > LGTM > > > > Reviewed-by: Greg Rose <gvrose8192@gmail.com> > > > > +simon > maybe missed it. Thanks, I see it now.
On Mon, Mar 18, 2019 at 09:39:00AM +0100, Simon Horman wrote: > On Sun, Mar 17, 2019 at 02:29:45PM +0000, Roi Dayan wrote: > > > > > > On 01/03/2019 02:26, Gregory Rose wrote: > > > > > > > > > On 2/28/2019 11:29 AM, Moshe Levi wrote: > > >> Signed-off-by: Moshe Levi <moshele@mellanox.com> > > >> --- > > >> lib/netdev-tc-offloads.c | 4 ++-- > > >> 1 files changed, 2 insertions(+), 2 deletions(-) > > >> > > >> diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c > > >> index 36bce15..e65f5b9 100644 > > >> --- a/lib/netdev-tc-offloads.c > > >> +++ b/lib/netdev-tc-offloads.c > > >> @@ -1015,12 +1015,12 @@ test_key_and_mask(struct match *match) > > >> key->nw_proto == IPPROTO_ICMPV6) { > > >> if (mask->tp_src) { > > >> VLOG_DBG_RL(&rl, > > >> - "offloading attribute icmp_type isn't supported"); > > >> + "offloading attribute icmpv6_type isn't supported"); > > >> return EOPNOTSUPP; > > >> } > > >> if (mask->tp_dst) { > > >> VLOG_DBG_RL(&rl, > > >> - "offloading attribute icmp_code isn't supported"); > > >> + "offloading attribute icmpv6_code isn't supported"); > > >> return EOPNOTSUPP; > > >> } > > >> } else if (key->dl_type == htons(OFP_DL_TYPE_NOT_ETH_TYPE)) { > > > > > > Nice catch... > > > > > > LGTM > > > > > > Reviewed-by: Greg Rose <gvrose8192@gmail.com> > > > > > > > +simon > > maybe missed it. > > Thanks, I see it now. Thanks again. I have pushed this change to: master, branch-2.11, branch-2.10, branch-2.9 and branch-2.8.
On Mon, Mar 18, 2019 at 02:59:19PM +0100, Simon Horman wrote: > On Mon, Mar 18, 2019 at 09:39:00AM +0100, Simon Horman wrote: > > On Sun, Mar 17, 2019 at 02:29:45PM +0000, Roi Dayan wrote: > > > > > > > > > On 01/03/2019 02:26, Gregory Rose wrote: > > > > > > > > > > > > On 2/28/2019 11:29 AM, Moshe Levi wrote: > > > >> Signed-off-by: Moshe Levi <moshele@mellanox.com> > > > >> --- > > > >> lib/netdev-tc-offloads.c | 4 ++-- > > > >> 1 files changed, 2 insertions(+), 2 deletions(-) > > > >> > > > >> diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c > > > >> index 36bce15..e65f5b9 100644 > > > >> --- a/lib/netdev-tc-offloads.c > > > >> +++ b/lib/netdev-tc-offloads.c > > > >> @@ -1015,12 +1015,12 @@ test_key_and_mask(struct match *match) > > > >> key->nw_proto == IPPROTO_ICMPV6) { > > > >> if (mask->tp_src) { > > > >> VLOG_DBG_RL(&rl, > > > >> - "offloading attribute icmp_type isn't supported"); > > > >> + "offloading attribute icmpv6_type isn't supported"); > > > >> return EOPNOTSUPP; > > > >> } > > > >> if (mask->tp_dst) { > > > >> VLOG_DBG_RL(&rl, > > > >> - "offloading attribute icmp_code isn't supported"); > > > >> + "offloading attribute icmpv6_code isn't supported"); > > > >> return EOPNOTSUPP; > > > >> } > > > >> } else if (key->dl_type == htons(OFP_DL_TYPE_NOT_ETH_TYPE)) { > > > > > > > > Nice catch... > > > > > > > > LGTM > > > > > > > > Reviewed-by: Greg Rose <gvrose8192@gmail.com> > > > > > > > > > > +simon > > > maybe missed it. > > > > Thanks, I see it now. > > Thanks again. > > I have pushed this change to: master, branch-2.11, branch-2.10, branch-2.9 > and branch-2.8. ..., and branch-2.7 and branch-2.6.
diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c index 36bce15..e65f5b9 100644 --- a/lib/netdev-tc-offloads.c +++ b/lib/netdev-tc-offloads.c @@ -1015,12 +1015,12 @@ test_key_and_mask(struct match *match) key->nw_proto == IPPROTO_ICMPV6) { if (mask->tp_src) { VLOG_DBG_RL(&rl, - "offloading attribute icmp_type isn't supported"); + "offloading attribute icmpv6_type isn't supported"); return EOPNOTSUPP; } if (mask->tp_dst) { VLOG_DBG_RL(&rl, - "offloading attribute icmp_code isn't supported"); + "offloading attribute icmpv6_code isn't supported"); return EOPNOTSUPP; } } else if (key->dl_type == htons(OFP_DL_TYPE_NOT_ETH_TYPE)) {
Signed-off-by: Moshe Levi <moshele@mellanox.com> --- lib/netdev-tc-offloads.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)