[PATCH rtems-lwip 3/3] rtemslwip: Use interrupt server for ISRs

Kinsey Moore kinsey.moore at oarcorp.com
Thu Mar 2 22:59:44 UTC 2023


lwIP requires locking to be available in its device ISRs. This is
incompatible with RTEMS ISRs, but could be worked around on single-core
systems. This moves lwIP device ISR execution to a task via the
interrupt server. All network device driver code should use the
interrupt server going forward.
---
 defs/common/lwip.json                |  1 +
 rtemslwip/beaglebone/netstart.c      |  5 +---
 rtemslwip/common/netstart_shared.c   | 40 ++++++++++++++++++++++++++++
 rtemslwip/include/netstart.h         |  2 ++
 rtemslwip/zynqmp/xil_shims.c         |  3 ++-
 rtemslwip/zynqmp_cfc400x/netstart.c  |  2 +-
 rtemslwip/zynqmp_hardware/netstart.c |  2 +-
 rtemslwip/zynqmp_qemu/netstart.c     |  2 +-
 uLan/ports/os/rtems/arch/sys_arch.c  |  2 --
 9 files changed, 49 insertions(+), 10 deletions(-)
 create mode 100644 rtemslwip/common/netstart_shared.c

diff --git a/defs/common/lwip.json b/defs/common/lwip.json
index a637c45..2dd5614 100644
--- a/defs/common/lwip.json
+++ b/defs/common/lwip.json
@@ -9,6 +9,7 @@
 		"uLan/ports/os/rtems/arch/sys_arch.c",
 		"rtemslwip/common/syslog.c",
 		"rtemslwip/common/rtems_lwip_io.c",
+		"rtemslwip/common/netstart_shared.c",
 		"rtemslwip/common/network_compat.c",
 		"rtemslwip/bsd_compat/netdb.c",
 		"rtemslwip/bsd_compat/ifaddrs.c",
diff --git a/rtemslwip/beaglebone/netstart.c b/rtemslwip/beaglebone/netstart.c
index 33384b6..a62e83d 100644
--- a/rtemslwip/beaglebone/netstart.c
+++ b/rtemslwip/beaglebone/netstart.c
@@ -25,9 +25,6 @@
  */
 
 #include <netstart.h>
-#include "soc_AM335x.h"
-#include "beaglebone.h"
-#include <lwip/tcpip.h>
 
 int start_networking(
   struct netif  *net_interface,
@@ -37,7 +34,7 @@ int start_networking(
   unsigned char *mac_ethernet_address
 )
 {
-  tcpip_init( NULL, NULL );
+  start_networking_shared();
 
   return 0;
 }
diff --git a/rtemslwip/common/netstart_shared.c b/rtemslwip/common/netstart_shared.c
new file mode 100644
index 0000000..216e5a1
--- /dev/null
+++ b/rtemslwip/common/netstart_shared.c
@@ -0,0 +1,40 @@
+/*
+ * Copyright (C) 2023 On-Line Applications Research Corporation (OAR)
+ * Written by Kinsey Moore <kinsey.moore at oarcorp.com>
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
+ * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include <netstart.h>
+#include <lwip/tcpip.h>
+
+rtems_status_code start_networking_shared(void)
+{
+  tcpip_init( NULL, NULL );
+  return rtems_interrupt_server_initialize(
+    1,
+    RTEMS_MINIMUM_STACK_SIZE,
+    RTEMS_DEFAULT_MODES,
+    RTEMS_DEFAULT_ATTRIBUTES,
+    NULL
+  );
+}
diff --git a/rtemslwip/include/netstart.h b/rtemslwip/include/netstart.h
index d5c58d4..807183a 100644
--- a/rtemslwip/include/netstart.h
+++ b/rtemslwip/include/netstart.h
@@ -38,4 +38,6 @@ int start_networking(
   unsigned char *mac_ethernet_address
 );
 
+rtems_status_code start_networking_shared(void);
+
 #endif
diff --git a/rtemslwip/zynqmp/xil_shims.c b/rtemslwip/zynqmp/xil_shims.c
index f0b1b9b..ec9ed12 100644
--- a/rtemslwip/zynqmp/xil_shims.c
+++ b/rtemslwip/zynqmp/xil_shims.c
@@ -67,7 +67,8 @@ BaseType_t xPortInstallInterruptHandler(
   void             *pvCallBackRef
 )
 {
-  rtems_status_code sc = rtems_interrupt_handler_install(
+  rtems_status_code sc = rtems_interrupt_server_handler_install(
+    RTEMS_INTERRUPT_SERVER_DEFAULT,
     ucInterruptID,
     "CGEM Handler",
     RTEMS_INTERRUPT_UNIQUE,
diff --git a/rtemslwip/zynqmp_cfc400x/netstart.c b/rtemslwip/zynqmp_cfc400x/netstart.c
index d19b36c..237133d 100644
--- a/rtemslwip/zynqmp_cfc400x/netstart.c
+++ b/rtemslwip/zynqmp_cfc400x/netstart.c
@@ -37,7 +37,7 @@ int start_networking(
   unsigned char *mac_ethernet_address
 )
 {
-  tcpip_init( NULL, NULL );
+  start_networking_shared();
 
   if ( !xemac_add(
     net_interface,
diff --git a/rtemslwip/zynqmp_hardware/netstart.c b/rtemslwip/zynqmp_hardware/netstart.c
index 01e392e..5d97ab2 100644
--- a/rtemslwip/zynqmp_hardware/netstart.c
+++ b/rtemslwip/zynqmp_hardware/netstart.c
@@ -37,7 +37,7 @@ int start_networking(
   unsigned char *mac_ethernet_address
 )
 {
-  tcpip_init( NULL, NULL );
+  start_networking_shared();
 
   if ( !xemac_add(
     net_interface,
diff --git a/rtemslwip/zynqmp_qemu/netstart.c b/rtemslwip/zynqmp_qemu/netstart.c
index d19b36c..237133d 100644
--- a/rtemslwip/zynqmp_qemu/netstart.c
+++ b/rtemslwip/zynqmp_qemu/netstart.c
@@ -37,7 +37,7 @@ int start_networking(
   unsigned char *mac_ethernet_address
 )
 {
-  tcpip_init( NULL, NULL );
+  start_networking_shared();
 
   if ( !xemac_add(
     net_interface,
diff --git a/uLan/ports/os/rtems/arch/sys_arch.c b/uLan/ports/os/rtems/arch/sys_arch.c
index 3a454b8..2651c9c 100644
--- a/uLan/ports/os/rtems/arch/sys_arch.c
+++ b/uLan/ports/os/rtems/arch/sys_arch.c
@@ -375,7 +375,6 @@ sys_arch_protect()
   sys_prot_t pval;
 
 #if RTEMS_SMP
-  pval = _Thread_Dispatch_disable();
   rtems_recursive_mutex_lock( &sys_arch_lock );
 #else
   rtems_interrupt_disable(pval);
@@ -388,7 +387,6 @@ sys_arch_unprotect(sys_prot_t pval)
 {
 #if RTEMS_SMP
   rtems_recursive_mutex_unlock( &sys_arch_lock );
-  _Thread_Dispatch_enable(pval);
 #else
   rtems_interrupt_enable(pval);
 #endif
-- 
2.30.2



More information about the devel mailing list