Re: [PATCH 2/4] staging: vt6656: vnt_beacon_xmit use extra_tx_headroom.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 18/05/2020 13:39, Dan Carpenter wrote:
On Sat, May 16, 2020 at 11:39:34AM +0100, Malcolm Priestley wrote:
Create room for vnt_tx_short_buf_head in sk_buff and vnt_tx_usb_header.

The struct ieee80211_mgmt is not longer in the header and is at
the initial skb->data point.

I feel like the SubmittingPatches guidelines on verb tenses and not
saying "this patch" or "I" has got everyone so worried that it's like
playing Taboo.  Do you mean that the struct moved before or after we
aply *this patch*?
The struct has not moved, before skb->data was copied on to the address along with the rest of frame.

So now struct needs to be at skb->data.


Signed-off-by: Malcolm Priestley <tvboxspy@xxxxxxxxx>

I can't understand the point of this patch at all.  Is it a fix or a
clean up?  If I had to guess from the subjec, I would say it's a
performance improvement but I don't know.
Well there is a performance improvement as there is only one buffer instead of two.

Mainly to bring into line with other drivers in the mac80211 tree there
is no need for a secondary buffer in driver.

Regards

Malcolm

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux