中断处理函数太长了吗？

只看该作者 · 2014-4-16 09:37

本帖最后由拨云人于 2014-4-16 11:35 编辑

主要问题:大家经常说，中断处理函数不要执行时间太长，  但我现在的工程本身是一个事件触发的系统，主程序只完成了一些初始化功能，然后就进入了睡眠并等待事件的发生（sleep and  wait for event）, 几乎所有要执行的功能，都是在中断里完成的，甚至在中断里调用了软件延时100ms ( delay_ms(100) )  这样的函数.
会不会造成一些不良后果呢？

有三个GPIO中断源，一个蓝牙协议栈中断,
系统可简述为3个模块：采集模块，存储模块（SD卡），BLE蓝牙模块
GPIO中断可分为：
1、由采集芯片产生，每4ms 产生一次，在中断处理函数里，我只是做了接收数据（通过SPI），并置了各种标志位.
2、按键中断，检测到按键按下之后，需要处理很多事情，例如 SD卡初始化，采集芯片初试化等，这个耗时会比较长.但是正常情况下，正在采集的时候，是把按键功能关闭的。
3、外部高电平中断，当检测到USB线的插入时，会在某个引脚检测到高电平，这时候我需要给SD卡重新上电，也就是在这里调用了 delay_ms(100)，这个中断处理过程也会很长。

中断2，3的处理时间都很长，但是正常情况下，如果正在采集（中断源1使能），那么中断源 2，3，都是处于关闭状态的。只有结束采集之后，才会打开。

在中断1的处理函数中，会根据接收到的数据，以及进入中断的次数，置很多的标志位.然后我的主循环就成了:
while(1)
{
sleep(); //每次从中断回来，都会往下执行一遍
if(xxx_flag==1)
   {
      xxx_flag= 0；
      do_someting;
   }; // 检测到某标志位，后..

if(xxx_flag==1)
   {
      xxx_flag= 0；
      do_someting;
   }; // 检测到另一个标志位后...

if(xxx_flag==1)
   {
      xxx_flag= 0；
      do_someting;
   };// 检测到另一个标志位后...
   ...
   ...
   ...
}
这样的结构，我都不忍直视了.:'(

只看该作者 · 2014-4-16 09:42

如果说你没有其他的中断的，其他程序时间或者优先级什么要求都不高的话，是可以的。但是最好在中断程序中不要做太多的工作的。可以搞个标志位，一些处理工作在主程序中做的

只看该作者 · 2014-4-16 09:49

最好是中断处理函数里只设置标志位，然后主循环里检查标志位再处理事件

1万 · 2014-4-16 10:08

如果有多个中断就不好

只看该作者 · 2014-4-16 10:11

楼主睡眠是为了低功耗？

只看该作者 · 2014-4-16 10:42

中断处理函数不要执行时间太长

这个“太长”到底是多长没人定义，一般视具体情况而定。
类似变频器的应用，可能整个ISR占了90%以上的CPU时间都是有可能的。

不过从楼主的ISR中有delay(100)来说只能说“呵呵”了

只看该作者 · 2014-4-16 11:31

本帖最后由拨云人于 2014-4-16 11:33 编辑

ayb_ice 发表于 2014-4-16 10:08
如果有多个中断就不好

有三个GPIO中断源，一个蓝牙协议栈中断,
系统可简述为3个模块：采集模块，存储模块（SD卡），BLE蓝牙模块
GPIO中断可分为：
1、由采集芯片产生，每4ms 产生一次，在中断处理函数里，我只是做了接收数据（通过SPI），并置了各种标志位.
2、按键中断，检测到按键按下之后，需要处理很多事情，例如 SD卡初始化，采集芯片初试化等，这个耗时会比较长.但是正常情况下，正在采集的时候，是把按键功能关闭的。
3、外部高电平中断，当检测到USB线的插入时，会在某个引脚检测到高电平，这时候我需要给SD卡重新上电，也就是在这里调用了 delay_ms(100)，这个中断处理过程也会很长。

中断2，3的处理时间都很长，但是正常情况下，如果正在采集（中断源1使能），那么中断源 2，3，都是处于关闭状态的。只有结束采集之后，才会打开。

在中断1的处理函数中，会根据接收到的数据，以及进入中断的次数，置很多的标志位.然后我的主循环就成了:
while(1)
{
      sleep(); //每次从中断回来，都会往下执行一遍
      if(xxx_flag==1)
      {
               xxx_flag= 0；
               do_someting;
      }; // 检测到某标志位，后..

      if(xxx_flag==1)
      {
               xxx_flag= 0；
               do_someting;
      }; // 检测到另一个标志位后...

      if(xxx_flag==1)
      {
               xxx_flag= 0；
               do_someting;
      };// 检测到另一个标志位后...
      ...
      ...
      ...
}
这样的结构，我都不忍直视了.:'(