Hi, On 17.02.2017 22:07, Pavel Belous wrote: > From: Pavel Belous <pavel.bel...@aquantia.com> > > We should allocate the number of pages based on the config parameter > AQ_CFG_RX_FRAME_MAX. > > Signed-off-by: Pavel Belous <pavel.bel...@aquantia.com>
> do { > if (spin_trylock(&ring->header.lock)) { > - frags = aq_nic_map_skb(self, skb, &buffers[0]); > + frags = aq_nic_map_skb(self, skb, buffers); > > - aq_ring_tx_append_buffs(ring, &buffers[0], frags); > + aq_ring_tx_append_buffs(ring, buffers, frags); > This change has nothing to do with what the commit message claims that the patch is about. Please dont mix fixes and totally unrelated cleanups in one patch. Regards, Lino