summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--components/drivers/include/ipc/ringbuffer.h31
-rw-r--r--components/drivers/ipc/ringbuffer.c20
-rw-r--r--examples/test/ringbuffer_test.c106
3 files changed, 130 insertions, 27 deletions
diff --git a/components/drivers/include/ipc/ringbuffer.h b/components/drivers/include/ipc/ringbuffer.h
index 97abf37097..fc2434cf4e 100644
--- a/components/drivers/include/ipc/ringbuffer.h
+++ b/components/drivers/include/ipc/ringbuffer.h
@@ -37,18 +37,15 @@ struct rt_ringbuffer
* | 0 | 1 | 2 | 3 | 4 | 5 | 6 ||| 0 | 1 | 2 | 3 | 4 | 5 | 6 | Empty
* +---+---+---+---+---+---+---+|+~~~+~~~+~~~+~~~+~~~+~~~+~~~+
* read_idx-^ ^-write_idx
- *
- * The tradeoff is we could only use 32KiB of buffer for 16 bit of index.
- * But it should be enough for most of the cases.
- *
- * Ref: http://en.wikipedia.org/wiki/Circular_buffer#Mirroring */
- rt_uint16_t read_mirror : 1;
- rt_uint16_t read_index : 15;
- rt_uint16_t write_mirror : 1;
- rt_uint16_t write_index : 15;
+ */
+
+ rt_uint32_t read_mirror : 1;
+ rt_uint32_t read_index : 31;
+ rt_uint32_t write_mirror : 1;
+ rt_uint32_t write_index : 31;
/* as we use msb of index as mirror bit, the size should be signed and
* could only be positive. */
- rt_int16_t buffer_size;
+ rt_int32_t buffer_size;
};
enum rt_ringbuffer_state
@@ -65,19 +62,19 @@ enum rt_ringbuffer_state
* Please note that the ring buffer implementation of RT-Thread
* has no thread wait or resume feature.
*/
-void rt_ringbuffer_init(struct rt_ringbuffer *rb, rt_uint8_t *pool, rt_int16_t size);
+void rt_ringbuffer_init(struct rt_ringbuffer *rb, rt_uint8_t *pool, rt_int32_t size);
void rt_ringbuffer_reset(struct rt_ringbuffer *rb);
-rt_size_t rt_ringbuffer_put(struct rt_ringbuffer *rb, const rt_uint8_t *ptr, rt_uint16_t length);
-rt_size_t rt_ringbuffer_put_force(struct rt_ringbuffer *rb, const rt_uint8_t *ptr, rt_uint16_t length);
+rt_size_t rt_ringbuffer_put(struct rt_ringbuffer *rb, const rt_uint8_t *ptr, rt_uint32_t length);
+rt_size_t rt_ringbuffer_put_force(struct rt_ringbuffer *rb, const rt_uint8_t *ptr, rt_uint32_t length);
rt_size_t rt_ringbuffer_putchar(struct rt_ringbuffer *rb, const rt_uint8_t ch);
rt_size_t rt_ringbuffer_putchar_force(struct rt_ringbuffer *rb, const rt_uint8_t ch);
-rt_size_t rt_ringbuffer_get(struct rt_ringbuffer *rb, rt_uint8_t *ptr, rt_uint16_t length);
+rt_size_t rt_ringbuffer_get(struct rt_ringbuffer *rb, rt_uint8_t *ptr, rt_uint32_t length);
rt_size_t rt_ringbuffer_peek(struct rt_ringbuffer *rb, rt_uint8_t **ptr);
rt_size_t rt_ringbuffer_getchar(struct rt_ringbuffer *rb, rt_uint8_t *ch);
rt_size_t rt_ringbuffer_data_len(struct rt_ringbuffer *rb);
#ifdef RT_USING_HEAP
-struct rt_ringbuffer* rt_ringbuffer_create(rt_uint16_t length);
+struct rt_ringbuffer* rt_ringbuffer_create(rt_uint32_t length);
void rt_ringbuffer_destroy(struct rt_ringbuffer *rb);
#endif
@@ -88,14 +85,14 @@ void rt_ringbuffer_destroy(struct rt_ringbuffer *rb);
*
* @return Buffer size.
*/
-rt_inline rt_uint16_t rt_ringbuffer_get_size(struct rt_ringbuffer *rb)
+rt_inline rt_uint32_t rt_ringbuffer_get_size(struct rt_ringbuffer *rb)
{
RT_ASSERT(rb != RT_NULL);
return rb->buffer_size;
}
/** return the size of empty space in rb */
-#define rt_ringbuffer_space_len(rb) ((rb)->buffer_size - (rt_int16_t)rt_ringbuffer_data_len(rb))
+#define rt_ringbuffer_space_len(rb) ((rb)->buffer_size - rt_ringbuffer_data_len(rb))
#ifdef __cplusplus
diff --git a/components/drivers/ipc/ringbuffer.c b/components/drivers/ipc/ringbuffer.c
index 90632f5884..3374b913b1 100644
--- a/components/drivers/ipc/ringbuffer.c
+++ b/components/drivers/ipc/ringbuffer.c
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2006-2021, RT-Thread Development Team
+ * Copyright (c) 2006-2023, RT-Thread Development Team
*
* SPDX-License-Identifier: Apache-2.0
*
@@ -37,7 +37,7 @@ rt_inline enum rt_ringbuffer_state rt_ringbuffer_status(struct rt_ringbuffer *rb
*/
void rt_ringbuffer_init(struct rt_ringbuffer *rb,
rt_uint8_t *pool,
- rt_int16_t size)
+ rt_int32_t size)
{
RT_ASSERT(rb != RT_NULL);
RT_ASSERT(size > 0);
@@ -63,9 +63,9 @@ RTM_EXPORT(rt_ringbuffer_init);
*/
rt_size_t rt_ringbuffer_put(struct rt_ringbuffer *rb,
const rt_uint8_t *ptr,
- rt_uint16_t length)
+ rt_uint32_t length)
{
- rt_uint16_t size;
+ rt_uint32_t size;
RT_ASSERT(rb != RT_NULL);
@@ -116,9 +116,9 @@ RTM_EXPORT(rt_ringbuffer_put);
*/
rt_size_t rt_ringbuffer_put_force(struct rt_ringbuffer *rb,
const rt_uint8_t *ptr,
- rt_uint16_t length)
+ rt_uint32_t length)
{
- rt_uint16_t space_length;
+ rt_uint32_t space_length;
RT_ASSERT(rb != RT_NULL);
@@ -177,7 +177,7 @@ RTM_EXPORT(rt_ringbuffer_put_force);
*/
rt_size_t rt_ringbuffer_get(struct rt_ringbuffer *rb,
rt_uint8_t *ptr,
- rt_uint16_t length)
+ rt_uint32_t length)
{
rt_size_t size;
@@ -192,7 +192,7 @@ rt_size_t rt_ringbuffer_get(struct rt_ringbuffer *rb,
/* less data */
if (size < length)
- length = (rt_uint16_t)size;
+ length = size;
if (rb->buffer_size - rb->read_index > length)
{
@@ -248,7 +248,7 @@ rt_size_t rt_ringbuffer_peek(struct rt_ringbuffer *rb, rt_uint8_t **ptr)
if ((rt_size_t)(rb->buffer_size - rb->read_index) > size)
{
- rb->read_index += (rt_uint16_t)size;
+ rb->read_index += size;
return size;
}
@@ -423,7 +423,7 @@ RTM_EXPORT(rt_ringbuffer_reset);
*
* @return Return a pointer to ring buffer object. When the return value is RT_NULL, it means this creation failed.
*/
-struct rt_ringbuffer *rt_ringbuffer_create(rt_uint16_t size)
+struct rt_ringbuffer *rt_ringbuffer_create(rt_uint32_t size)
{
struct rt_ringbuffer *rb;
rt_uint8_t *pool;
diff --git a/examples/test/ringbuffer_test.c b/examples/test/ringbuffer_test.c
new file mode 100644
index 0000000000..d3037bf6a4
--- /dev/null
+++ b/examples/test/ringbuffer_test.c
@@ -0,0 +1,106 @@
+/*
+ * Copyright (c) 2006-2021, RT-Thread Development Team
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * Change Logs:
+ * Date Author Notes
+ * 2023-02-07 ZXY the first version
+ */
+#include <rtthread.h>
+#include <rtdevice.h>
+#include <string.h>
+#include <ipc/ringbuffer.h>
+
+#define RING_BUFFER_LEN 8
+static struct ringbuffer *rb;
+static char *str = "Hello, World new ringbuffer32";
+typedef struct rb_example {
+ rt_uint32_t a;
+ rt_uint32_t b;
+ rt_uint32_t c;
+} rb_example_t;
+
+
+int ringbuffer_example(void)
+{
+ rb_example_t data = {
+ .a = 1,
+ .b = 2,
+ };
+
+ struct rt_ringbuffer * rb = rt_ringbuffer_create(sizeof(rb_example_t) * 2);
+ RT_ASSERT(rb != RT_NULL);
+
+ rt_kprintf("Put data to ringbuffer, a: %d b: %d size: %d\n", data.a, data.b, sizeof(data));
+ rt_ringbuffer_put(rb, (rt_uint8_t *)&data, sizeof(data));
+
+
+ rb_example_t recv_data;
+ rt_size_t recv = rt_ringbuffer_get(rb, (rt_uint8_t *)&recv_data, sizeof(recv_data));
+ RT_ASSERT(recv == sizeof(recv_data));
+ rt_kprintf("Get data from ringbuffer, a: %d b: %d size: %d\n", recv_data.a, recv_data.b, sizeof(recv_data));
+
+ return 0;
+}
+MSH_CMD_EXPORT(ringbuffer_example, ringbuffer example);
+
+
+int ringbuffer_force_example(void)
+{
+ uint8_t test[6] = {1,2,3,4,5,6};
+ struct rt_ringbuffer * rb;
+ rb = rt_ringbuffer_create(4);
+ RT_ASSERT(rb != RT_NULL);
+
+ rt_kprintf("Put data to ringbuffer, %d %d %d %d %d %d\n", test[0],test[1],test[2],test[3],test[4],test[5]);
+ rt_ringbuffer_put_force(rb, (rt_uint8_t *)&test, sizeof(test));
+
+
+ uint8_t recv_data[4]={0};
+ rt_ringbuffer_get(rb, (rt_uint8_t *)&recv_data, sizeof(test));
+ rt_kprintf("Get data from ringbuffer, %d %d %d %d\n", recv_data[0],recv_data[1],recv_data[2],recv_data[3]);
+ rt_kprintf("write mirror: %d read mirror: %d\n", rb->write_mirror,rb->read_mirror);
+ return 0;
+}
+MSH_CMD_EXPORT(ringbuffer_force_example, ringbuffer example);
+
+static void consumer_thread_entry(void *arg)
+{
+ char ch;
+ while (1)
+ {
+ if (1 == rt_ringbuffer_getchar(rb, &ch))
+ {
+ rt_kprintf("[Consumer] <- %c\n", ch);
+ }
+ rt_thread_mdelay(500);
+ }
+}
+static void ringbuffer_sample(int argc, char** argv)
+{
+ rt_thread_t tid;
+ rt_uint16_t i = 0;
+ rb = rt_ringbuffer_create(RING_BUFFER_LEN);
+ if (rb == RT_NULL)
+ {
+ rt_kprintf("Can't create ringbffer");
+ return;
+ }
+ tid = rt_thread_create("consumer", consumer_thread_entry, RT_NULL,
+ 1024, RT_THREAD_PRIORITY_MAX/3, 20);
+ if (tid == RT_NULL)
+ {
+ rt_ringbuffer_destroy(rb);
+ }
+ rt_thread_startup(tid);
+ while (str[i] != '\0')
+ {
+ rt_kprintf("[Producer] -> %c\n", str[i]);
+ rt_ringbuffer_putchar(rb, str[i++]);
+ rt_thread_mdelay(500);
+ }
+ rt_thread_delete(tid);
+ rt_ringbuffer_destroy(rb);
+}
+MSH_CMD_EXPORT(ringbuffer_sample, Start a producer and a consumer with a ringbuffer);