osmocom
/
libosmocore
mirror of https://gerrit.osmocom.org/libosmocore
11
0
Fork 0
Code Issues Releases Wiki Activity

sercomm: make rx msg size configurable per instance 

Change-Id: I835506e26e83232e1e7290c0da04d389c8d7fb40
This commit is contained in:
Harald Welte 2017-05-01 00:19:13 +02:00
parent 1358836282
commit f6adcd7dd6
2 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
include/osmocom/core/sercomm.h
View File

@ -39,6 +39,7 @@ struct osmo_sercomm_inst {
/* receive side */
struct {
dlci_cb_t dlci_handler[_SC_DLCI_MAX];
unsigned int msg_size;
struct msgb *msg;
int state;
uint8_t dlci;

10
src/sercomm.c
View File

@ -28,7 +28,7 @@
#ifdef HOST_BUILD
# define SERCOMM_RX_MSG_SIZE 2048
# define DEFAULT_RX_MSG_SIZE 2048
# ifndef ARRAY_SIZE
# define ARRAY_SIZE(x) (sizeof(x)/sizeof(x[0]))
# endif
@ -39,7 +39,7 @@ static inline void sercomm_unlock(unsigned long __attribute__((unused)) *flags)
#else
# define SERCOMM_RX_MSG_SIZE 256
# define DEFAULT_RX_MSG_SIZE 256
# include <debug.h>
# include <osmocom/core/linuxlist.h>
# include <asm/system.h>
@ -87,6 +87,8 @@ void osmo_sercomm_init(struct osmo_sercomm_inst *sercomm)
INIT_LLIST_HEAD(&sercomm->tx.dlci_queues[i]);
sercomm->rx.msg = NULL;
if (!sercomm->rx.msg_size)
sercomm->rx.msg_size = DEFAULT_RX_MSG_SIZE;
sercomm->initialized = 1;
/* set up the echo dlci */
@ -262,12 +264,12 @@ int osmo_sercomm_drv_rx_char(struct osmo_sercomm_inst *sercomm, uint8_t ch)
* which means that any data structures we use need to be for
* our exclusive access */
if (!sercomm->rx.msg)
sercomm->rx.msg = osmo_sercomm_alloc_msgb(SERCOMM_RX_MSG_SIZE);
sercomm->rx.msg = osmo_sercomm_alloc_msgb(sercomm->rx.msg_size);
if (msgb_tailroom(sercomm->rx.msg) == 0) {
//cons_puts("sercomm_drv_rx_char() overflow!\n");
msgb_free(sercomm->rx.msg);
sercomm->rx.msg = osmo_sercomm_alloc_msgb(SERCOMM_RX_MSG_SIZE);
sercomm->rx.msg = osmo_sercomm_alloc_msgb(sercomm->rx.msg_size);
sercomm->rx.state = RX_ST_WAIT_START;
return 0;
}