RE: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet

From: Madalin-Cristian Bucur
Date: Wed Nov 09 2016 - 12:16:33 EST


> From: Madalin-Cristian Bucur
> Sent: Monday, November 07, 2016 5:43 PM
>
> > From: David Miller [mailto:davem@xxxxxxxxxxxxx]
> > Sent: Thursday, November 03, 2016 9:58 PM
> >
> > From: Madalin Bucur <madalin.bucur@xxxxxxx>
> > Date: Wed, 2 Nov 2016 22:17:26 +0200
> >
> > > This introduces the Freescale Data Path Acceleration Architecture
<snip>

> > > + int numstats = sizeof(struct rtnl_link_stats64) / sizeof(u64);
> > ...
> > > + cpustats = (u64 *)&percpu_priv->stats;
> > > +
> > > + for (j = 0; j < numstats; j++)
> > > + netstats[j] += cpustats[j];
> >
> > This is a memcpy() on well-typed datastructures which requires no
> > casting or special handling whatsoever, so use memcpy instead of
> > needlessly open coding the operation.
>
> Will fix.

Took a second look at this, it's not copying but adding the percpu
statistics into consolidated results.

Madalin