gnrc_netif: introduce GNRC_NETIF_STACKSIZE_DEFAULT

sys/net/gnrc/netif/init_devs/auto_init_atwinc15x0.c

@@ -23,12 +23,13 @@
 #include "atwinc15x0.h"
 #include "atwinc15x0_params.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
 /**
  * @brief   Define stack parameters for the MAC layer thread
  * @{
  */
-#define ATWINC15X0_MAC_STACKSIZE    (THREAD_STACKSIZE_DEFAULT)
+#define ATWINC15X0_MAC_STACKSIZE    (GNRC_NETIF_STACKSIZE_DEFAULT)
 #ifndef ATWINC15X0_MAC_PRIO
 #define ATWINC15X0_MAC_PRIO         (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/auto_init_cc110x.c

@@ -23,28 +23,15 @@
 #include "cc1xxx_common.h"
 #include "cc110x_params.h"
 #include "log.h"
-#include "msg.h"
+#include "include/init_devs.h"
-#include "net/gnrc/netif/conf.h"    /* <- GNRC_NETIF_MSG_QUEUE_SIZE */
 
 #define ENABLE_DEBUG 0
 #include "debug.h"
 
-#ifndef CC110X_EXTRA_STACKSIZE
-/**
- * @brief   Additional stack size required by the driver
- *
- * With increasing of GNRC_NETIF_MSG_QUEUE_SIZE the required stack size
- * increases as well. A queue size of 8 messages works with default stack size,
- * so we increase the stack by `sizeof(msg_t)` for each additional element
- */
-#define CC110X_EXTRA_STACKSIZE          ((GNRC_NETIF_MSG_QUEUE_SIZE - 8) * sizeof(msg_t))
-#endif
-
 /**
  * @brief   Calculate the stack size for the MAC layer thread(s)
  */
-#define CC110X_MAC_STACKSIZE            (THREAD_STACKSIZE_DEFAULT + \
+#define CC110X_MAC_STACKSIZE            (GNRC_NETIF_STACKSIZE_DEFAULT + \
-                                        CC110X_EXTRA_STACKSIZE + \
                                         DEBUG_EXTRA_STACKSIZE)
 #ifndef CC110X_MAC_PRIO
 /**

sys/net/gnrc/netif/init_devs/auto_init_dose.c

@@ -22,12 +22,13 @@
 #include "dose.h"
 #include "dose_params.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
 /**
  * @brief   Define stack parameters for the MAC layer thread
  * @{
  */
-#define DOSE_MAC_STACKSIZE (THREAD_STACKSIZE_DEFAULT + DEBUG_EXTRA_STACKSIZE)
+#define DOSE_MAC_STACKSIZE (GNRC_NETIF_STACKSIZE_DEFAULT + DEBUG_EXTRA_STACKSIZE)
 #ifndef DOSE_MAC_PRIO
 #define DOSE_MAC_PRIO      (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/auto_init_enc28j60.c

@@ -23,12 +23,13 @@
 #include "enc28j60.h"
 #include "enc28j60_params.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
 /**
  * @brief   Define stack parameters for the MAC layer thread
  * @{
  */
-#define ENC28J60_MAC_STACKSIZE   (THREAD_STACKSIZE_DEFAULT)
+#define ENC28J60_MAC_STACKSIZE   (GNRC_NETIF_STACKSIZE_DEFAULT)
 #ifndef ENC28J60_MAC_PRIO
 #define ENC28J60_MAC_PRIO        (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/auto_init_encx24j600.c

@@ -22,12 +22,13 @@
 #include "encx24j600.h"
 #include "encx24j600_params.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
 /**
  * @brief   Define stack parameters for the MAC layer thread
  * @{
  */
-#define ENCX24J600_MAC_STACKSIZE    (THREAD_STACKSIZE_DEFAULT + DEBUG_EXTRA_STACKSIZE)
+#define ENCX24J600_MAC_STACKSIZE    (GNRC_NETIF_STACKSIZE_DEFAULT + DEBUG_EXTRA_STACKSIZE)
 #ifndef ENCX24J600_MAC_PRIO
 #define ENCX24J600_MAC_PRIO         (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/auto_init_esp_eth.c

@@ -19,6 +19,7 @@
 #include "esp_eth_params.h"
 #include "esp_eth_netdev.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
 static gnrc_netif_t _netif;
 

sys/net/gnrc/netif/init_devs/auto_init_ethos.c

@@ -23,6 +23,7 @@
 #include "ethos_params.h"
 #include "periph/uart.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
 #define ETHOS_NUM ARRAY_SIZE(ethos_params)
 
@@ -37,7 +38,7 @@ static gnrc_netif_t _netif[ETHOS_NUM];
  * @brief   Define stack parameters for the MAC layer thread
  * @{
  */
-#define ETHOS_MAC_STACKSIZE (THREAD_STACKSIZE_DEFAULT + DEBUG_EXTRA_STACKSIZE)
+#define ETHOS_MAC_STACKSIZE (GNRC_NETIF_STACKSIZE_DEFAULT + DEBUG_EXTRA_STACKSIZE)
 #ifndef ETHOS_MAC_PRIO
 #define ETHOS_MAC_PRIO      (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/auto_init_netdev_tap.c

@@ -21,8 +21,9 @@
 #include "debug.h"
 #include "netdev_tap_params.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
-#define TAP_MAC_STACKSIZE           (THREAD_STACKSIZE_DEFAULT + DEBUG_EXTRA_STACKSIZE)
+#define TAP_MAC_STACKSIZE           (GNRC_NETIF_STACKSIZE_DEFAULT + DEBUG_EXTRA_STACKSIZE)
 #define TAP_MAC_PRIO                (GNRC_NETIF_PRIO)
 
 static netdev_tap_t netdev_tap[NETDEV_TAP_MAX];

sys/net/gnrc/netif/init_devs/auto_init_nrf24l01p_ng.c

@@ -24,6 +24,8 @@
 #include "log.h"
 #include "msg.h"
 #include "net/gnrc/netif/conf.h"
+#include "include/init_devs.h"
+
 #define ENABLE_DEBUG 0
 #include "debug.h"
 
@@ -42,7 +44,7 @@
 /**
  * @brief   Calculate the stack size for the MAC layer thread(s)
  */
-#define NRF24L01P_NG_MAC_STACKSIZE          (THREAD_STACKSIZE_DEFAULT + \
+#define NRF24L01P_NG_MAC_STACKSIZE          (GNRC_NETIF_STACKSIZE_DEFAULT + \
                                             NRF24L01P_NG_EXTRA_STACKSIZE + \
                                             DEBUG_EXTRA_STACKSIZE)
 #ifndef NRF24L01P_NG_MAC_PRIO

sys/net/gnrc/netif/init_devs/auto_init_sam0_eth.c

@@ -19,9 +19,10 @@
 
 #include "net/gnrc/netif/ethernet.h"
 #include "sam0_eth_netdev.h"
+#include "include/init_devs.h"
 
 static netdev_t sam0eth;
-static char stack[THREAD_STACKSIZE_DEFAULT];
+static char stack[GNRC_NETIF_STACKSIZE_DEFAULT];
 static gnrc_netif_t _netif;
 
 void auto_init_sam0_eth(void)
@@ -29,7 +30,7 @@ void auto_init_sam0_eth(void)
     /* setup netdev device */
     sam0_eth_setup(&sam0eth);
     /* initialize netdev <-> gnrc adapter state */
-    gnrc_netif_ethernet_create(&_netif, stack, THREAD_STACKSIZE_DEFAULT,
+    gnrc_netif_ethernet_create(&_netif, stack, GNRC_NETIF_STACKSIZE_DEFAULT,
                                GNRC_NETIF_PRIO, "sam0_eth", &sam0eth);
 }
 

sys/net/gnrc/netif/init_devs/auto_init_slipdev.c

@@ -21,6 +21,7 @@
 #include "board.h"
 #include "net/gnrc/netif/raw.h"
 #include "net/gnrc.h"
+#include "include/init_devs.h"
 
 #include "slipdev.h"
 #include "slipdev_params.h"
@@ -31,7 +32,7 @@
  * @brief   Define stack parameters for the MAC layer thread
  * @{
  */
-#define SLIPDEV_STACKSIZE       (THREAD_STACKSIZE_DEFAULT)
+#define SLIPDEV_STACKSIZE       (GNRC_NETIF_STACKSIZE_DEFAULT)
 #ifndef SLIPDEV_PRIO
 #define SLIPDEV_PRIO            (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/auto_init_stm32_eth.c

@@ -9,9 +9,10 @@
 
 #include "stm32_eth.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
 static netdev_t stm32eth;
-static char stack[THREAD_STACKSIZE_DEFAULT];
+static char stack[GNRC_NETIF_STACKSIZE_DEFAULT];
 static gnrc_netif_t _netif;
 
 void auto_init_stm32_eth(void)
@@ -19,7 +20,7 @@ void auto_init_stm32_eth(void)
   /* setup netdev device */
   stm32_eth_netdev_setup(&stm32eth);
         /* initialize netdev <-> gnrc adapter state */
-  gnrc_netif_ethernet_create(&_netif, stack, THREAD_STACKSIZE_DEFAULT, GNRC_NETIF_PRIO, "stm32_eth",
+  gnrc_netif_ethernet_create(&_netif, stack, GNRC_NETIF_STACKSIZE_DEFAULT, GNRC_NETIF_PRIO, "stm32_eth",
                              &stm32eth);
 }
 /** @} */

sys/net/gnrc/netif/init_devs/auto_init_sx126x.c

@@ -24,6 +24,7 @@
 #include "net/gnrc/netif/lorawan_base.h"
 #include "net/gnrc/netif/raw.h"
 #include "net/gnrc.h"
+#include "include/init_devs.h"
 
 #include "sx126x.h"
 #include "sx126x_params.h"
@@ -36,7 +37,7 @@
 /**
  * @brief   Define stack parameters for the MAC layer thread
  */
-#define SX126X_STACKSIZE            (THREAD_STACKSIZE_DEFAULT)
+#define SX126X_STACKSIZE            (GNRC_NETIF_STACKSIZE_DEFAULT)
 #ifndef SX126X_PRIO
 #define SX126X_PRIO                 (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/auto_init_sx127x.c

@@ -24,6 +24,7 @@
 #include "net/gnrc/netif/lorawan_base.h"
 #include "net/gnrc/netif/raw.h"
 #include "net/gnrc.h"
+#include "include/init_devs.h"
 
 #include "sx127x.h"
 #include "sx127x_params.h"
@@ -36,7 +37,7 @@
 /**
  * @brief   Define stack parameters for the MAC layer thread
  */
-#define SX127X_STACKSIZE           (THREAD_STACKSIZE_DEFAULT)
+#define SX127X_STACKSIZE           (GNRC_NETIF_STACKSIZE_DEFAULT)
 #ifndef SX127X_PRIO
 #define SX127X_PRIO                (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/auto_init_usbus_cdc_ecm.c

@@ -22,6 +22,7 @@
 #include "log.h"
 #include "usb/usbus/cdc/ecm.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
 /**
  * @brief global cdc ecm object, declared in the usb auto init file
@@ -32,7 +33,7 @@ extern usbus_cdcecm_device_t cdcecm;
  * @brief   Define stack parameters for the MAC layer thread
  * @{
  */
-#define CDCECM_MAC_STACKSIZE (THREAD_STACKSIZE_DEFAULT)
+#define CDCECM_MAC_STACKSIZE (GNRC_NETIF_STACKSIZE_DEFAULT)
 #ifndef CDCECM_MAC_PRIO
 #define CDCECM_MAC_PRIO      (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/auto_init_w5100.c

@@ -21,12 +21,13 @@
 #include "w5100.h"
 #include "w5100_params.h"
 #include "net/gnrc/netif/ethernet.h"
+#include "include/init_devs.h"
 
 /**
  * @brief   Define stack parameters for the MAC layer thread
  * @{
  */
-#define MAC_STACKSIZE   (THREAD_STACKSIZE_DEFAULT)
+#define MAC_STACKSIZE   (GNRC_NETIF_STACKSIZE_DEFAULT)
 #define MAC_PRIO        (GNRC_NETIF_PRIO)
 /*** @} */
 

sys/net/gnrc/netif/init_devs/auto_init_xbee.c

@@ -24,6 +24,7 @@
 #include "gnrc_netif_xbee.h"
 #include "xbee.h"
 #include "xbee_params.h"
+#include "include/init_devs.h"
 
 /**
  * @brief   Calculate the number of configured XBee devices
@@ -33,7 +34,7 @@
 /**
  * @brief   Define stack parameters for the MAC layer thread
  */
-#define XBEE_MAC_STACKSIZE           (THREAD_STACKSIZE_DEFAULT)
+#define XBEE_MAC_STACKSIZE           (GNRC_NETIF_STACKSIZE_DEFAULT)
 #ifndef XBEE_MAC_PRIO
 #define XBEE_MAC_PRIO                (GNRC_NETIF_PRIO)
 #endif

sys/net/gnrc/netif/init_devs/include/init_devs.h

@@ -20,29 +20,41 @@
 #define INIT_DEVS_H
 
 #include "thread.h"
+#include "msg.h"
+#include "net/gnrc/netif/conf.h"    /* <- GNRC_NETIF_MSG_QUEUE_SIZE */
 
 #ifdef __cplusplus
 extern "C" {
 #endif
 
+/**
+ * @brief   stack size of a netif thread
+ *
+ *          Message queue was previously allocated on the stack, reduce
+ *          stack size by default msg queue size to keep the RAM use the same
+ */
+#ifndef GNRC_NETIF_STACKSIZE_DEFAULT
+#define GNRC_NETIF_STACKSIZE_DEFAULT    (THREAD_STACKSIZE_DEFAULT - 128)
+#endif
+
 /**
  * @brief   extra stack size if ieee802154 security is enabled
  *
  * You may increase this value if you experience a stack overflow
  * with IEEE 802.15.4 security enabled.
  */
+#ifdef MODULE_IEEE802154_SECURITY
 #define IEEE802154_SECURITY_EXTRA_STACKSIZE (128)
+#else
+#define IEEE802154_SECURITY_EXTRA_STACKSIZE (0)
+#endif
 
 #ifndef IEEE802154_STACKSIZE_DEFAULT
-#ifdef MODULE_IEEE802154_SECURITY
-#define IEEE802154_STACKSIZE_DEFAULT    (THREAD_STACKSIZE_DEFAULT + \
-                                        IEEE802154_SECURITY_EXTRA_STACKSIZE)
-#else
 /**
  * @brief   stack size of an ieee802154 device
  */
-#define IEEE802154_STACKSIZE_DEFAULT    (THREAD_STACKSIZE_DEFAULT)
+#define IEEE802154_STACKSIZE_DEFAULT    (GNRC_NETIF_STACKSIZE_DEFAULT + \
-#endif
+                                         IEEE802154_SECURITY_EXTRA_STACKSIZE)
 #endif
 
 #ifdef __cplusplus