All of lore.kernel.org
 help / color / mirror / Atom feed
* [Xenomai] Rework of mlockall and dlopen handling
@ 2013-04-23 15:33 Jan Kiszka
  2013-04-23 18:30 ` Gilles Chanteperdrix
  0 siblings, 1 reply; 4+ messages in thread
From: Jan Kiszka @ 2013-04-23 15:33 UTC (permalink / raw)
  To: Xenomai

Hi,

I've coded down what we discussed and pushed it into 'for-upstream'.
This does not yet remove redundant mlockall from included userspace
code or change the docs, but the core mechanisms are ready. Patch list
below summarizes the steps.

Jan


Jan Kiszka (4):
      Invoke mlockall on every skin library initialization
      posix: Control auto-shadowing via environment variable
      Remove mlockall alert handler
      Drop --enable-dlopen, disable optimized TLS support by default

 configure.in               |   70 +++----------------------------------------
 include/asm-generic/bind.h |   10 ------
 src/skins/common/bind.c    |   24 ++++-----------
 src/skins/posix/init.c     |   52 +++++++++++---------------------
 src/skins/psos+/init.c     |   15 +---------
 src/skins/uitron/init.c    |   15 +---------
 6 files changed, 31 insertions(+), 155 deletions(-)


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Xenomai] Rework of mlockall and dlopen handling
  2013-04-23 15:33 [Xenomai] Rework of mlockall and dlopen handling Jan Kiszka
@ 2013-04-23 18:30 ` Gilles Chanteperdrix
  2013-04-23 18:36   ` Jan Kiszka
  0 siblings, 1 reply; 4+ messages in thread
From: Gilles Chanteperdrix @ 2013-04-23 18:30 UTC (permalink / raw)
  To: Jan Kiszka; +Cc: Xenomai

On 04/23/2013 05:33 PM, Jan Kiszka wrote:

> Hi,
> 
> I've coded down what we discussed and pushed it into 'for-upstream'.
> This does not yet remove redundant mlockall from included userspace
> code or change the docs, but the core mechanisms are ready. Patch list
> below summarizes the steps.


I would have been a bit less extreme, and simply inverted the default
value of --enable-dlopen-skins and --enable-*-auto-mlockall, but what
you did is fine by me.

Maybe we could keep --disable-dlopen-skins though?


-- 
                                                                Gilles.


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Xenomai] Rework of mlockall and dlopen handling
  2013-04-23 18:30 ` Gilles Chanteperdrix
@ 2013-04-23 18:36   ` Jan Kiszka
  2013-04-23 19:04     ` Gilles Chanteperdrix
  0 siblings, 1 reply; 4+ messages in thread
From: Jan Kiszka @ 2013-04-23 18:36 UTC (permalink / raw)
  To: Gilles Chanteperdrix; +Cc: Xenomai

On 2013-04-23 20:30, Gilles Chanteperdrix wrote:
> On 04/23/2013 05:33 PM, Jan Kiszka wrote:
> 
>> Hi,
>>
>> I've coded down what we discussed and pushed it into 'for-upstream'.
>> This does not yet remove redundant mlockall from included userspace
>> code or change the docs, but the core mechanisms are ready. Patch list
>> below summarizes the steps.
> 
> 
> I would have been a bit less extreme, and simply inverted the default
> value of --enable-dlopen-skins and --enable-*-auto-mlockall, but what
> you did is fine by me.
> 
> Maybe we could keep --disable-dlopen-skins though?

...and translate it to --with-__thread? Or what should be the semantics?

Jan

-- 
Siemens AG, Corporate Technology, CT RTC ITP SDP-DE
Corporate Competence Center Embedded Linux


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Xenomai] Rework of mlockall and dlopen handling
  2013-04-23 18:36   ` Jan Kiszka
@ 2013-04-23 19:04     ` Gilles Chanteperdrix
  0 siblings, 0 replies; 4+ messages in thread
From: Gilles Chanteperdrix @ 2013-04-23 19:04 UTC (permalink / raw)
  To: Jan Kiszka; +Cc: Xenomai

On 04/23/2013 08:36 PM, Jan Kiszka wrote:

> On 2013-04-23 20:30, Gilles Chanteperdrix wrote:
>> On 04/23/2013 05:33 PM, Jan Kiszka wrote:
>>
>>> Hi,
>>>
>>> I've coded down what we discussed and pushed it into 'for-upstream'.
>>> This does not yet remove redundant mlockall from included userspace
>>> code or change the docs, but the core mechanisms are ready. Patch list
>>> below summarizes the steps.
>>
>>
>> I would have been a bit less extreme, and simply inverted the default
>> value of --enable-dlopen-skins and --enable-*-auto-mlockall, but what
>> you did is fine by me.
>>
>> Maybe we could keep --disable-dlopen-skins though?
> 
> ...and translate it to --with-__thread? Or what should be the semantics?


Yes, you are right, it is useless.

-- 
                                                                Gilles.


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-04-23 19:04 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-23 15:33 [Xenomai] Rework of mlockall and dlopen handling Jan Kiszka
2013-04-23 18:30 ` Gilles Chanteperdrix
2013-04-23 18:36   ` Jan Kiszka
2013-04-23 19:04     ` Gilles Chanteperdrix

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.