All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Marc Strämke" <marc.straemke@eltropuls.de>
To: xenomai@xenomai.org
Subject: Re: [Xenomai] rtnet locking/socket SKBs
Date: Thu, 04 Jun 2015 13:09:08 +0200	[thread overview]
Message-ID: <2172332.WGhck7t4nv@eltrolinux48.eltroad.eltropuls> (raw)
In-Reply-To: <20150604105616.GS27570@hermes.click-hack.org>

Gilles,

Am Donnerstag, 4. Juni 2015, 12:56:16 schrieb Gilles Chanteperdrix:
> The previous rtnet design was converted to reference counts globally
> when integrating into xenomai 3, some details may still need fixing.
> The old design was different but lead to the close syscall looping
> for ever and blocking your application if anything went wrong, so,
> this is an attempt to avoid this issue.
Yes I do understand this. I am just trying to figure out what the right fix is. 
If I just disable incrementing the fd reference count (see attached patch) 
AF_PACKET works as it should.  From my current understanding of the code the 
reference counting for the SKB Pools attached to a socket is redundant, but 
I've only closely inspected AF_PACKET not the other sockets in the IP stack.

> 
> > They will never get cleared if the application crashes If I am
> > not mistaken.
> 
> Once again: when an application crashes, the file descriptors are
> automatically closed by Xenomai, you do not need to care about this
> case.
What I am seeing is that when the reference count is above 1 when the 
application closes/crashes, the close operation of the rtdm socket never get's 
called, neither when doing an explicit close system call on the socket which 
still had a reference count above 1.

When I disable the skb pool reference counting for AF_PACKET it basically 
works as it should. When the application closes the socket or crashes the 
close op get's called and the skb's release. 

Marc

-- 
Dipl.-Ing. Marc Strämke
Geschäftsführer / CEO

ELTROPULS Anlagenbau GmbH
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Proof-of-concept-disable-locking-in-af_packet-skb-po.patch
Type: text/x-patch
Size: 1318 bytes
Desc: not available
URL: <http://xenomai.org/pipermail/xenomai/attachments/20150604/b098fe8a/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: This is a digitally signed message part.
URL: <http://xenomai.org/pipermail/xenomai/attachments/20150604/b098fe8a/attachment.sig>

  reply	other threads:[~2015-06-04 11:09 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-03 12:56 [Xenomai] Problem with RTNET Packet Socket (or RTDM in general) in Xenomai 3 Marc Strämke
2015-06-03 13:02 ` Gilles Chanteperdrix
2015-06-03 17:09   ` Marc Strämke
2015-06-04 10:41     ` [Xenomai] rtnet locking/socket SKBs Marc Strämke
2015-06-04 10:56       ` Gilles Chanteperdrix
2015-06-04 11:09         ` Marc Strämke [this message]
2015-06-04 11:12           ` [Xenomai] [PATCH] Proof of concept: disable locking in af_packet skb pool Marc Strämke
2015-06-04 14:15             ` Gilles Chanteperdrix
2015-06-04 11:20           ` [Xenomai] rtnet locking/socket SKBs Gilles Chanteperdrix
2015-06-04 11:31           ` Gilles Chanteperdrix
2015-06-04 12:35             ` Marc Strämke
2015-06-04 12:37               ` Gilles Chanteperdrix
2015-06-04 12:40                 ` Marc Strämke
2015-06-04 14:07               ` Gilles Chanteperdrix
2015-09-27 21:32       ` Gilles Chanteperdrix
2015-09-29 21:59         ` Gilles Chanteperdrix
2015-09-30 14:33           ` Marc Strämke
2015-09-30 14:41             ` Gilles Chanteperdrix
2015-09-30 14:44               ` Marc Strämke

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2172332.WGhck7t4nv@eltrolinux48.eltroad.eltropuls \
    --to=marc.straemke@eltropuls.de \
    --cc=xenomai@xenomai.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.