ronny meeus wrote:
> Hello
>
> I did some further debugging on the problem described below and I made
> some progress.
> At creation time of the message queue using:
> q_create("TEST",0,Q_NOLIMIT|Q_PRIOR,&qid);
> a chunk of message buffers (64) is allocated and added to the free
> message list of the queue (queue->freeq).
> Once the message queue is deleted, the messages are added into the
> global psosmbufq.
> During the q_create/q_delete loop, the memory pool get depleted since
> the number of messages in the psosmbufq keeps on increasing all the time.
>
> In my opinion if the Q_PRIBUF is not set during queue creation, the
> "psosmbufq" has to be used to allocate/release from/to.
> This also implies that the local freeq in the queue object is not used
> in this mode anymore.
> Each time a message needs to be send, in function get_mbuf it is simply
> taken from the psosmbufq. In case this would be empty, a feed_pool
> operation is called on it.
>
> static psosmbuf_t *get_mbuf(psosqueue_t *queue, u_long msglen)
> {
> psosmbuf_t *mbuf = NULL;
>
> if (testbits(queue->synchbase.status, Q_NOCACHE)) {
> mbuf =
> (psosmbuf_t *) xnmalloc(sizeof(*mbuf) + msglen -
> sizeof(mbuf->data));
>
> if (mbuf)
> inith(&mbuf->link);
> } else {
> xnholder_t *holder = NULL;
> if (testbits(queue->synchbase.status, Q_SHAREDINIT)) {
> holder = getq(&psosmbufq);
> if (!holder) {
> feed_pool(&psoschunkq,
> &psosmbufq,PSOS_QUEUE_MIN_ALLOC,queue->maxlen);
> holder = getq(&psosmbufq);
> }
> } else {
> holder = getq(&queue->freeq);
> if (!holder && testbits(queue->synchbase.status, Q_INFINITE)) {
> feed_pool(&queue->chunkq,&queue->freeq,
> PSOS_QUEUE_MIN_ALLOC,queue->maxlen);
> holder = getq(&queue->freeq);
> }
> }
> if (holder)
> mbuf = link2psosmbuf(holder);
> }
> return mbuf;
> }
>
> I have adapted the code accordingly and rerun my tests. Now it runs forever.
> (Of course I also did changes in the code to create and delete a queue.)
>
> Now the question is: Is my understanding correct? If it is, the flag
> Q_SHAREDINIT would be better renamed to Q_SHAREDMSGS.
>
> Please share your thoughts.
A patch would be better than some fancy HTML colouring, if you intend
your fix to be reviewed/integrated.
--
Gilles.
_______________________________________________
Xenomai-help mailing list
[email protected]
https://mail.gna.org/listinfo/xenomai-help
