uorb:Fixed O_DIRECT flag triggering rpmsg cross-core

Signed-off-by: likun17 <[email protected]>

Author: Otpvondoiats

drivers/sensors/sensor.c

@@ -613,7 +613,7 @@ static int sensor_open(FAR struct file *filep)
       goto errout_with_lock;
     }
 
-  if (lower->ops->open)
+  if ((filep->f_oflags & O_DIRECT) == 0 && lower->ops->open)
     {
       ret = lower->ops->open(lower, filep);
       if (ret < 0)
@@ -622,31 +622,28 @@ static int sensor_open(FAR struct file *filep)
         }
     }
 
-  if ((filep->f_oflags & O_DIRECT) == 0)
+  if (filep->f_oflags & O_RDOK)
     {
-      if (filep->f_oflags & O_RDOK)
+      if (upper->state.nsubscribers == 0 && lower->ops->activate)
         {
-          if (upper->state.nsubscribers == 0 && lower->ops->activate)
+          ret = lower->ops->activate(lower, filep, true);
+          if (ret < 0)
             {
-              ret = lower->ops->activate(lower, filep, true);
-              if (ret < 0)
-                {
-                  goto errout_with_open;
-                }
+              goto errout_with_open;
             }
-
-          user->role |= SENSOR_ROLE_RD;
-          upper->state.nsubscribers++;
         }
 
-      if (filep->f_oflags & O_WROK)
+      user->role |= SENSOR_ROLE_RD;
+      upper->state.nsubscribers++;
+    }
+
+  if (filep->f_oflags & O_WROK)
+    {
+      user->role |= SENSOR_ROLE_WR;
+      upper->state.nadvertisers++;
+      if (filep->f_oflags & SENSOR_PERSIST)
         {
-          user->role |= SENSOR_ROLE_WR;
-          upper->state.nadvertisers++;
-          if (filep->f_oflags & SENSOR_PERSIST)
-            {
-              lower->persist = true;
-            }
+          lower->persist = true;
         }
     }
 
@@ -695,7 +692,7 @@ static int sensor_close(FAR struct file *filep)
   int ret = 0;
 
   nxrmutex_lock(&upper->lock);
-  if (lower->ops->close)
+  if ((filep->f_oflags & O_DIRECT) == 0 && lower->ops->close)
     {
       ret = lower->ops->close(lower, filep);
       if (ret < 0)
@@ -705,21 +702,18 @@ static int sensor_close(FAR struct file *filep)
         }
     }
 
-  if ((filep->f_oflags & O_DIRECT) == 0)
+  if (filep->f_oflags & O_RDOK)
     {
-      if (filep->f_oflags & O_RDOK)
+      upper->state.nsubscribers--;
+      if (upper->state.nsubscribers == 0 && lower->ops->activate)
         {
-          upper->state.nsubscribers--;
-          if (upper->state.nsubscribers == 0 && lower->ops->activate)
-            {
-              lower->ops->activate(lower, filep, false);
-            }
+          lower->ops->activate(lower, filep, false);
         }
+    }
 
-      if (filep->f_oflags & O_WROK)
-        {
-          upper->state.nadvertisers--;
-        }
+  if (filep->f_oflags & O_WROK)
+    {
+      upper->state.nadvertisers--;
     }
 
   list_delete(&user->node);