Re: remove unused definition in if_vxlan.c

2022-04-02 Thread Theo de Raadt
Not now.

The ABI/API is locked for the upcoming release.

Denis Fondras  wrote:

> VXLANMTU appears nowhere else in the codebase.
> 
> OK to remove it ?
> 
> 
> Index: if_vxlan.c
> ===
> RCS file: /cvs/src/sys/net/if_vxlan.c,v
> retrieving revision 1.90
> diff -u -p -r1.90 if_vxlan.c
> --- if_vxlan.c26 Feb 2022 04:46:34 -  1.90
> +++ if_vxlan.c2 Apr 2022 15:17:49 -
> @@ -67,8 +67,6 @@
>  /*
>   * The protocol.
>   */
> -
> -#define VXLANMTU 1492
>  #define VXLAN_PORT   4789
>  
>  struct vxlan_header {
> 



remove unused definition in if_vxlan.c

2022-04-02 Thread Denis Fondras
VXLANMTU appears nowhere else in the codebase.

OK to remove it ?


Index: if_vxlan.c
===
RCS file: /cvs/src/sys/net/if_vxlan.c,v
retrieving revision 1.90
diff -u -p -r1.90 if_vxlan.c
--- if_vxlan.c  26 Feb 2022 04:46:34 -  1.90
+++ if_vxlan.c  2 Apr 2022 15:17:49 -
@@ -67,8 +67,6 @@
 /*
  * The protocol.
  */
-
-#define VXLANMTU   1492
 #define VXLAN_PORT 4789
 
 struct vxlan_header {