On 22/11/18 14:33, Guenter Roeck wrote:
drivers/net/ethernet/ibm/ibmvnic.c: In function 'ibmvnic_xmit':
drivers/net/ethernet/ibm/ibmvnic.c:789:40: error: implicit declaration
of function 'skb_vlan_tag_present'
Caused by 5f51bb855fa1 ("ibmvnic: fix accelerated VLAN handling").
Problem is missing include of linux/if_vlan.h, which was added upstream
with commit 6052d5e2a1961b ("ibmvnic: Insert header on VLAN tagged
received frame").
I have no idea if the offending patch fixes 6052d5e2a1961b, if both are
independent, if both are needed, or if none is needed.
Copying the authors of both patches.
Both address different parts of the code within the same VLAN
functionality, patch 5f51bb855fa1 (or upstream e84b47941e15) is making a
change on the transmit side, and the patch I sent upstream
(6052d5e2a196) addresses an issue when the system firmware strips the
VLAN tag on the receive side.
In the context of having proper VLAN support for the ibmvnic driver I
would say both are needed, although potentially there are other patches
needed for this functionality to work properly, for example upstream
commit da75e3b6a029 (ibmvnic: Account for VLAN tag in L2 Header
descriptor) and upstream commit 8dff66cc4195 (ibmvnic: Account for VLAN
tag in L2 Header descriptor).
Best regards,
Murilo Vicentini
