[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170519100428.GC21003@localhost>
Date: Fri, 19 May 2017 12:04:28 +0200
From: Miroslav Lichvar <mlichvar@...hat.com>
To: Willem de Bruijn <willemdebruijn.kernel@...il.com>
Cc: Network Development <netdev@...r.kernel.org>,
Richard Cochran <richardcochran@...il.com>,
Willem de Bruijn <willemb@...gle.com>
Subject: Re: [PATCH v5 net-next 4/7] net: add new control message for
incoming HW-timestamped packets
On Thu, May 18, 2017 at 04:20:53PM -0400, Willem de Bruijn wrote:
> On Thu, May 18, 2017 at 10:07 AM, Miroslav Lichvar <mlichvar@...hat.com> wrote:
> > +SOF_TIMESTAMPING_OPT_PKTINFO:
> > +
> > + Enable the SCM_TIMESTAMPING_PKTINFO control message for incoming
> > + packets with hardware timestamps. The message contains struct
> > + scm_ts_pktinfo, which supplies the index of the real interface which
> > + received the packet and its length at layer 2. A valid (non-zero)
> > + interface index will be returned only if CONFIG_NET_RX_BUSY_POLL is
> > + enabled and the driver is using NAPI.
>
> It is probably good to explicitly call out that the remaining two fields
> are reserved and undefined. To stress that applications cannot be
> overly pedantic and start failing if these become non-zero.
Ok. I'm adding "The struct contains also two other fields, but they
are reserved and undefined".
--
Miroslav Lichvar
Powered by blists - more mailing lists