Re: [Linuxptp-devel] [PATCH RFC 12/30] interface: Introduce a method to initialize the time stamping label.
On 2/11/2020 6:04 AM, Richard Cochran wrote: > In many cases, the time stamping label will be the same as the name of > the interface. In order to prevent users from open coding the logic that > initializes the label from the interface name, this patch add an > appropriate method. > > Signed-off-by: Richard Cochran > --- > interface.c | 7 +++ > interface.h | 6 ++ > 2 files changed, 13 insertions(+) > > diff --git a/interface.c b/interface.c > index 460ceb8..662552d 100644 > --- a/interface.c > +++ b/interface.c > @@ -6,6 +6,13 @@ > */ > #include "interface.h" > > +void interface_ensure_tslabel(struct interface *iface) > +{ > + if (!iface->ts_label[0]) { The original code did "if (label[0] == '\0')" but this is equivalent. Ok. > + strncpy(iface->ts_label, iface->name, MAX_IFNAME_SIZE); > + } > +} > + > int interface_get_tsinfo(struct interface *iface) > { > return sk_get_ts_info(iface->ts_label, &iface->ts_info); > diff --git a/interface.h b/interface.h > index 05cfb10..371185d 100644 > --- a/interface.h > +++ b/interface.h > @@ -24,6 +24,12 @@ struct interface { > struct sk_ts_info ts_info; > }; > > +/** > + * Ensures that an interface has a proper time stamping label. > + * @param iface The interface of interest. > + */ > +void interface_ensure_tslabel(struct interface *iface); > + > /** > * Populate the time stamping information of a given interface. > * @param iface The interface of interest. > ___ Linuxptp-devel mailing list Linuxptp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxptp-devel
Re: [Linuxptp-devel] [PATCH RFC 12/30] interface: Introduce a method to initialize the time stamping label.
On 2/11/2020 6:04 AM, Richard Cochran wrote: > In many cases, the time stamping label will be the same as the name of > the interface. In order to prevent users from open coding the logic that > initializes the label from the interface name, this patch add an > appropriate method. > > Signed-off-by: Richard Cochran Reviewed-by: Jacob Keller > --- > interface.c | 7 +++ > interface.h | 6 ++ > 2 files changed, 13 insertions(+) > > diff --git a/interface.c b/interface.c > index 460ceb8..662552d 100644 > --- a/interface.c > +++ b/interface.c > @@ -6,6 +6,13 @@ > */ > #include "interface.h" > > +void interface_ensure_tslabel(struct interface *iface) > +{ > + if (!iface->ts_label[0]) { > + strncpy(iface->ts_label, iface->name, MAX_IFNAME_SIZE); > + } > +} > + > int interface_get_tsinfo(struct interface *iface) > { > return sk_get_ts_info(iface->ts_label, &iface->ts_info); > diff --git a/interface.h b/interface.h > index 05cfb10..371185d 100644 > --- a/interface.h > +++ b/interface.h > @@ -24,6 +24,12 @@ struct interface { > struct sk_ts_info ts_info; > }; > > +/** > + * Ensures that an interface has a proper time stamping label. > + * @param iface The interface of interest. > + */ > +void interface_ensure_tslabel(struct interface *iface); > + > /** > * Populate the time stamping information of a given interface. > * @param iface The interface of interest. > ___ Linuxptp-devel mailing list Linuxptp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxptp-devel
[Linuxptp-devel] [PATCH RFC 12/30] interface: Introduce a method to initialize the time stamping label.
In many cases, the time stamping label will be the same as the name of the interface. In order to prevent users from open coding the logic that initializes the label from the interface name, this patch add an appropriate method. Signed-off-by: Richard Cochran --- interface.c | 7 +++ interface.h | 6 ++ 2 files changed, 13 insertions(+) diff --git a/interface.c b/interface.c index 460ceb8..662552d 100644 --- a/interface.c +++ b/interface.c @@ -6,6 +6,13 @@ */ #include "interface.h" +void interface_ensure_tslabel(struct interface *iface) +{ + if (!iface->ts_label[0]) { + strncpy(iface->ts_label, iface->name, MAX_IFNAME_SIZE); + } +} + int interface_get_tsinfo(struct interface *iface) { return sk_get_ts_info(iface->ts_label, &iface->ts_info); diff --git a/interface.h b/interface.h index 05cfb10..371185d 100644 --- a/interface.h +++ b/interface.h @@ -24,6 +24,12 @@ struct interface { struct sk_ts_info ts_info; }; +/** + * Ensures that an interface has a proper time stamping label. + * @param iface The interface of interest. + */ +void interface_ensure_tslabel(struct interface *iface); + /** * Populate the time stamping information of a given interface. * @param iface The interface of interest. -- 2.20.1 ___ Linuxptp-devel mailing list Linuxptp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxptp-devel