ethernetif_input中的LOCK_TCPIP_CORE是否必要

Jokeoo

MCU:H743VI
Lwip：2.0.2

问题
1.st早期的lwip例程中ethernetif_input里面，在low_level_input()前添加了LOCK_TCPIP_CORE()，并在netif->input()后释放，这一步是否必要？在接收udp报文时有时候会莫名其妙的ping不通，导致数据丢失，把这个锁去掉就好了。

1. void ethernetif_input( void const *argument )
   
2. {
   
3.   struct pbuf *p;
   
4.   struct netif *netif = (struct netif *) argument;
   
5.   
   
6.   for( ;; )
   
7.   {
   
8.     if (osSemaphoreWait( RxPktSemaphore, TIME_WAITING_FOR_INPUT)==osOK)
   
9.     {
   
10.       do
    
11.       {
    
12.         //LOCK_TCPIP_CORE();
    
13.         p = low_level_input( netif );
    
14.         if (p != NULL)
    
15.         {
    
16.           if (netif->input( p, netif) != ERR_OK )
    
17.           {
    
18.             pbuf_free(p);
    
19.           }
    
20.         }
    
21.         //UNLOCK_TCPIP_CORE();
    
22.       }while(p!=NULL);
    
23.     }
    
24.   }
    
25. }

复制代码

2.另外偶尔出现83848和lwip初始化完成后板子莫名其妙直接复位，怀疑是mpu问题，一直没找到原因。而且同样的固件和板子，不一定什么时候会出现问题。

Jokeoo

MPU配置如下

1. static void MPU_Config( void )
   
2. {
   
3.   MPU_Region_InitTypeDef MPU_InitStruct;
   
4. 
   
5.   /* Disable the MPU */
   
6.   HAL_MPU_Disable();
   
7. 
   
8.   /* Configure the MPU attributes as Device not cacheable
   
9.      for ETH DMA descriptors */
   
10.   MPU_InitStruct.Enable           = MPU_REGION_ENABLE;
    
11.   MPU_InitStruct.BaseAddress      = 0x30040000;
    
12.   MPU_InitStruct.Size             = MPU_REGION_SIZE_256B;
    
13.   MPU_InitStruct.AccessPermission = MPU_REGION_FULL_ACCESS;
    
14.   MPU_InitStruct.IsBufferable     = MPU_ACCESS_BUFFERABLE;
    
15.   MPU_InitStruct.IsCacheable      = MPU_ACCESS_NOT_CACHEABLE;
    
16.   MPU_InitStruct.IsShareable      = MPU_ACCESS_NOT_SHAREABLE;
    
17.   MPU_InitStruct.Number           = MPU_REGION_NUMBER0;
    
18.   MPU_InitStruct.TypeExtField     = MPU_TEX_LEVEL0;
    
19.   MPU_InitStruct.SubRegionDisable = 0x00;
    
20.   MPU_InitStruct.DisableExec      = MPU_INSTRUCTION_ACCESS_ENABLE;
    
21. 
    
22.   HAL_MPU_ConfigRegion( &MPU_InitStruct );
    
23. 
    
24.   /* Configure the MPU attributes as Cacheable write through
    
25.      for LwIP RAM heap which contains the Tx buffers */
    
26.   MPU_InitStruct.Enable           = MPU_REGION_ENABLE;
    
27.   MPU_InitStruct.BaseAddress      = 0x30044000;
    
28.   MPU_InitStruct.Size             = MPU_REGION_SIZE_16KB;
    
29.   MPU_InitStruct.AccessPermission = MPU_REGION_FULL_ACCESS;
    
30.   MPU_InitStruct.IsBufferable     = MPU_ACCESS_NOT_BUFFERABLE;
    
31.   MPU_InitStruct.IsCacheable      = MPU_ACCESS_CACHEABLE;
    
32.   MPU_InitStruct.IsShareable      = MPU_ACCESS_NOT_SHAREABLE;
    
33.   MPU_InitStruct.Number           = MPU_REGION_NUMBER1;
    
34.   MPU_InitStruct.TypeExtField     = MPU_TEX_LEVEL0;
    
35.   MPU_InitStruct.SubRegionDisable = 0x00;
    
36.   MPU_InitStruct.DisableExec      = MPU_INSTRUCTION_ACCESS_ENABLE;
    
37. 
    
38.   HAL_MPU_ConfigRegion( &MPU_InitStruct );
    
39. 
    
40.   /* Configure the MPU attributes as Cacheable write through
    
41.      for LwIP RAM heap which contains the Tx buffers */
    
42.   MPU_InitStruct.Enable           = MPU_REGION_ENABLE;
    
43.   MPU_InitStruct.BaseAddress      = 0x30020000;
    
44.   MPU_InitStruct.Size             = MPU_REGION_SIZE_128KB;
    
45.   MPU_InitStruct.AccessPermission = MPU_REGION_FULL_ACCESS;
    
46.   MPU_InitStruct.IsBufferable     = MPU_ACCESS_NOT_BUFFERABLE;
    
47.   MPU_InitStruct.IsCacheable      = MPU_ACCESS_CACHEABLE;
    
48.   MPU_InitStruct.IsShareable      = MPU_ACCESS_NOT_SHAREABLE;
    
49.   MPU_InitStruct.Number           = MPU_REGION_NUMBER2;
    
50.   MPU_InitStruct.TypeExtField     = MPU_TEX_LEVEL0;
    
51.   MPU_InitStruct.SubRegionDisable = 0x00;
    
52.   MPU_InitStruct.DisableExec      = MPU_INSTRUCTION_ACCESS_ENABLE;
    
53. 
    
54.   HAL_MPU_ConfigRegion( &MPU_InitStruct );
    
55. 
    
56.   /* Configure the MPU attributes as Cacheable write through
    
57.      for LwIP RAM heap which contains the Tx buffers */
    
58.   MPU_InitStruct.Enable           = MPU_REGION_ENABLE;
    
59.   MPU_InitStruct.BaseAddress      = 0x30000000;
    
60.   MPU_InitStruct.Size             = MPU_REGION_SIZE_128KB;
    
61.   MPU_InitStruct.AccessPermission = MPU_REGION_FULL_ACCESS;
    
62.   MPU_InitStruct.IsBufferable     = MPU_ACCESS_NOT_BUFFERABLE;
    
63.   MPU_InitStruct.IsCacheable      = MPU_ACCESS_CACHEABLE;
    
64.   MPU_InitStruct.IsShareable      = MPU_ACCESS_NOT_SHAREABLE;
    
65.   MPU_InitStruct.Number           = MPU_REGION_NUMBER3;
    
66.   MPU_InitStruct.TypeExtField     = MPU_TEX_LEVEL0;
    
67.   MPU_InitStruct.SubRegionDisable = 0x00;
    
68.   MPU_InitStruct.DisableExec      = MPU_INSTRUCTION_ACCESS_ENABLE;
    
69. 
    
70.   HAL_MPU_ConfigRegion( &MPU_InitStruct );
    
71. 
    
72.   /* Enable the MPU */
    
73.   HAL_MPU_Enable( MPU_PRIVILEGED_DEFAULT );
    
74. }

复制代码

eric2013

我用的是CMSIS-Driver里的：

1. static void
   
2. ethernetif_input(struct netif *netif)
   
3. {
   
4.   struct ethernetif *eth = netif->state;
   
5.   struct pbuf *p;
   
6. 
   
7.   /* move received packet into a new pbuf */
   
8.   sys_sem_wait (&eth->sem);
   
9.   p = low_level_input(netif);
   
10.   sys_sem_signal (&eth->sem);
    
11.   /* if no packet could be read, silently ignore this */
    
12.   if (p != NULL) {
    
13.     /* pass all packets to ethernet_input, which decides what packets it supports */
    
14.     if (netif->input(p, netif) != ERR_OK) {
    
15.       LWIP_DEBUGF(NETIF_DEBUG, ("ethernetif_input: IP input error\n"));
    
16.       pbuf_free(p);
    
17.       p = NULL;
    
18.     }
    
19.     return;
    
20.   }
    
21.   eth->rx_event = false;
    
22. }

复制代码

LwIP网络教程开始更新，使用MDK的RTE环境开发，配套RTX5和FreeRTOS两个版本，更新至第7章（2020-05-03）
http://www.armbbs.cn/forum.php?m ... 5874&fromuid=58
(出处: 硬汉嵌入式论坛)

Jokeoo

eric2013 发表于 2020-10-20 01:15
我用的是CMSIS-Driver里的：

这种相当于对netif加锁吧
这个锁在别的地方还有调用吗？
是不是在netif_add的时候创建的信号量？

如果没有这个线程锁，tftp的时候总会出现数据错误。