[rtems commit] rtems: Move _Partition_Is_buffer_on_boundary()

Sebastian Huber sebh at rtems.org
Tue Nov 24 06:40:41 UTC 2020


Module:    rtems
Branch:    master
Commit:    a61e221d04a1df5d113606ad2c2c978e2c578655
Changeset: http://git.rtems.org/rtems/commit/?id=a61e221d04a1df5d113606ad2c2c978e2c578655

Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
Date:      Wed Oct 14 10:26:53 2020 +0200

rtems: Move _Partition_Is_buffer_on_boundary()

It is used only by rtems_partition_return_buffer().    Make the PTCB the
first parameter.  Rename to _Partition_Is_address_on_buffer_boundary().

---

 cpukit/include/rtems/rtems/partimpl.h | 22 ----------------------
 cpukit/rtems/src/partreturnbuffer.c   | 17 ++++++++++++++++-
 2 files changed, 16 insertions(+), 23 deletions(-)

diff --git a/cpukit/include/rtems/rtems/partimpl.h b/cpukit/include/rtems/rtems/partimpl.h
index 23d5422..0d15d1e 100644
--- a/cpukit/include/rtems/rtems/partimpl.h
+++ b/cpukit/include/rtems/rtems/partimpl.h
@@ -19,7 +19,6 @@
 #define _RTEMS_RTEMS_PARTIMPL_H
 
 #include <rtems/rtems/partdata.h>
-#include <rtems/score/address.h>
 #include <rtems/score/chainimpl.h>
 #include <rtems/score/objectimpl.h>
 
@@ -35,27 +34,6 @@ extern "C" {
  * @{
  */
 
-/**
- *  @brief Checks whether is on a valid buffer boundary for the_partition.
- *
- *  This function returns TRUE if the_buffer is on a valid buffer
- *  boundary for the_partition, and FALSE otherwise.
- */
-RTEMS_INLINE_ROUTINE bool _Partition_Is_buffer_on_boundary (
-  void              *the_buffer,
-  Partition_Control *the_partition
-)
-{
-  intptr_t offset;
-
-  offset = _Addresses_Subtract(
-    the_buffer,
-    the_partition->starting_address
-  );
-
-  return ((offset % the_partition->buffer_size) == 0);
-}
-
 RTEMS_INLINE_ROUTINE bool _Partition_Is_buffer_size_aligned(
   uint32_t buffer_size
 )
diff --git a/cpukit/rtems/src/partreturnbuffer.c b/cpukit/rtems/src/partreturnbuffer.c
index 08cbf9e..6d7b71a 100644
--- a/cpukit/rtems/src/partreturnbuffer.c
+++ b/cpukit/rtems/src/partreturnbuffer.c
@@ -24,6 +24,21 @@
 #include <rtems/score/address.h>
 #include <rtems/score/chainimpl.h>
 
+static bool _Partition_Is_address_on_buffer_boundary(
+  const Partition_Control *the_partition,
+  const void              *the_buffer
+)
+{
+  intptr_t offset;
+
+  offset = _Addresses_Subtract(
+    the_buffer,
+    the_partition->starting_address
+  );
+
+  return ( offset % the_partition->buffer_size ) == 0;
+}
+
 static bool _Partition_Is_address_a_buffer_begin(
    const Partition_Control *the_partition,
    const void              *the_buffer
@@ -36,7 +51,7 @@ static bool _Partition_Is_address_a_buffer_begin(
   ending   = _Addresses_Add_offset( starting, the_partition->length );
 
   return _Addresses_Is_in_range( the_buffer, starting, ending )
-    && _Partition_Is_buffer_on_boundary( the_buffer, the_partition );
+    && _Partition_Is_address_on_buffer_boundary( the_partition, the_buffer );
 }
 
 static void _Partition_Free_buffer(



More information about the vc mailing list