[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <MW4PR11MB57763617C193C52BB0C05E43FD129@MW4PR11MB5776.namprd11.prod.outlook.com>
Date: Thu, 17 Mar 2022 16:11:51 +0000
From: "Drewek, Wojciech" <wojciech.drewek@...el.com>
To: David Ahern <dsahern@...il.com>,
Harald Welte <laforge@...monks.org>
CC: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"stephen@...workplumber.org" <stephen@...workplumber.org>
Subject: RE: [PATCH iproute2-next v5 1/2] ip: GTP support in ip link
> -----Original Message-----
> From: David Ahern <dsahern@...il.com>
> Sent: czwartek, 17 marca 2022 15:48
> To: Harald Welte <laforge@...monks.org>; Drewek, Wojciech <wojciech.drewek@...el.com>
> Cc: netdev@...r.kernel.org; stephen@...workplumber.org
> Subject: Re: [PATCH iproute2-next v5 1/2] ip: GTP support in ip link
>
> On 3/17/22 8:02 AM, Harald Welte wrote:
> > Hi Wojciech, David,
> >
> > On Thu, Mar 17, 2022 at 11:11:40AM +0000, Drewek, Wojciech wrote:
> >>> as a u32 does that mean more roles might get added? Seems like this
> >>> should have a attr to string converter that handles future additions.
> >>
> >> I think no more roles are expected but we can ask Harald.
> >
> > I also don't currently know of any situation where we would want to add
> > more roles.
> >
>
> Better safe than giving users wrong information. I would prefer if this
> attribute to string conversion handle the 2 known roles and return
> "unknown" if a new value pops up.
Sure, I will include this in the 7th version.
Powered by blists - more mailing lists