<div dir="ltr"><div dir="ltr">Hi Kamal,<div><br></div><div>See <a href="https://lists.zx2c4.com/pipermail/wireguard/2020-April/005237.html" target="_blank">https://lists.zx2c4.com/pipermail/wireguard/2020-April/005237.html</a></div><div><br></div><div>There's no need to CC Pablo or upstream on matters like this. His patch and the backport of it to 5.4 and 5.5 are entirely okay and even expected. The responsibility to keep the compat layer up to date is ours entirely. You can CC <a href="mailto:wireguard@lists.zx2c4.com" target="_blank">wireguard@lists.zx2c4.com</a> for these things too, where a conversation on this issue took place.</div><div><br></div><div>Jason</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 1, 2020 at 9:50 AM Kamal Mostafa <<a href="mailto:kamal@canonical.com" target="_blank">kamal@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Jason, etc.-<div><br></div><div>This patch from linux-stable v5.4.29 will break our Ubuntu Focal dkms build of wireguard:</div><div><br></div><div><font face="monospace" size="1">f8c60f7a0051 net: Fix CONFIG_NET_CLS_ACT=n and CONFIG_NFT_FWD_NETDEV={y, m} build<br></font></div><div><br></div><div>Like so:</div><div><br></div><div><font face="monospace" size="1"><<DKMSDIR>>/build/wireguard/1.0.20200330/build/compat/compat.h:1029:20: error: redefinition of 'skb_reset_redirect'<br> 1029 | static inline void skb_reset_redirect(struct sk_buff *skb)</font></div><div><font face="monospace" size="1">./include/linux/skbuff.h:4538:20: note: previous definition of 'skb_reset_redirect' was here<br> 4538 | static inline void skb_reset_redirect(struct sk_buff *skb)<br></font></div><div><font face="monospace" size="1"><br></font></div><div><font face="monospace" size="1"><<DKMSDIR>>/build/wireguard/1.0.20200330/build/compat/compat.h: In function 'skb_reset_redirect':<br><<DKMSDIR>>/build/wireguard/1.0.20200330/build/compat/compat.h:1032:2: error: implicit declaration of function 'skb_reset_tc'<br></font></div><div><font face="monospace" size="1"><br></font></div><div><font face="monospace" size="1"><br></font></div><div>The mainline commit it is based upon ...</div><div><font face="monospace" size="1"><br></font></div><div><font face="monospace" size="1">2c64605b590e net: Fix CONFIG_NET_CLS_ACT=n and CONFIG_NFT_FWD_NETDEV={y, m} build<br></font></div><div><br></div><div>... touches drivers/net/wireguard/queueing.h but that change is (of course) omitted from the v5.4.29 backport patch.  Will you be fixing wireguard's compat.h for this?</div><div><br></div><div> -Kamal (Canonical Kernel Team)</div><div><br></div></div>
</blockquote></div><div><br></div>
</div>