From: Eric Dumazet <dada1@cosmosbay.com>
To: Davide Libenzi <davidel@xmailserver.org>
Cc: Kyle Moffett <mrmacman_g4@mac.com>,
Alan Cox <alan@lxorguk.ukuu.org.uk>,
Ulrich Drepper <drepper@redhat.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Linus Torvalds <torvalds@linux-foundation.org>,
Andrew Morton <akpm@linux-foundation.org>,
Ingo Molnar <mingo@elte.hu>
Subject: Re: [patch 7/8] fdmap v2 - implement sys_socket2
Date: Thu, 07 Jun 2007 22:05:12 +0200 [thread overview]
Message-ID: <466864F8.2050903@cosmosbay.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0706071243090.6445@alien.or.mcafeemobile.com>
Davide Libenzi a écrit :
> On Thu, 7 Jun 2007, Eric Dumazet wrote:
>
>> Davide, are you sure we want FIFO for non sequential allocations ?
>>
>> This tends to use all the fmap slots, and not very cache friendly
>> if an app does a lot of [open(),...,close()] things. We already got a
>> perf drop because of RCUification of file freeing (FIFO mode instead
>> of LIFO given by kmalloc()/kfree())
>>
>> If the idea behind this FIFO was security (ie not easy for an app to predict
>> next glibc file handle), we/glibc might use yet another FD_SECUREMODE flag,
>> wich ORed with O_NONSEQFD would ask to fdmap_newfd() to take the tail of
>> fmap->slist, not head.
>
> Uli, would it be OK to rely only on base randomization and use a LIFO
> instead? We have base randomization, plus LIFO does not mean strictly
> sequential like legacy allocator, just more compatc and cache friendly.
>
I am afraid randomization wont really work if /sbin/init or /bin/bash for
example uses one (or more) unseq fd :
The 'random base' will be propagated at fork()/exec() time ?
next prev parent reply other threads:[~2007-06-07 20:06 UTC|newest]
Thread overview: 129+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-06 22:30 [patch 7/8] fdmap v2 - implement sys_socket2 Davide Libenzi
2007-06-06 22:44 ` David Miller
2007-06-06 22:52 ` Davide Libenzi
2007-06-06 22:57 ` David Miller
2007-06-06 22:57 ` Ulrich Drepper
2007-06-06 23:02 ` David Miller
2007-06-06 22:59 ` Alan Cox
2007-06-06 22:58 ` Ulrich Drepper
2007-06-06 23:04 ` Davide Libenzi
2007-06-06 23:08 ` David Miller
2007-06-06 23:19 ` Alan Cox
2007-06-06 23:22 ` Ulrich Drepper
2007-06-07 10:04 ` Alan Cox
2007-06-07 11:59 ` Kyle Moffett
2007-06-07 13:12 ` Eric Dumazet
2007-06-07 15:51 ` Davide Libenzi
2007-06-07 19:49 ` Davide Libenzi
2007-06-07 20:02 ` Ulrich Drepper
2007-06-07 20:05 ` Eric Dumazet [this message]
2007-06-07 20:18 ` Ulrich Drepper
2007-06-07 21:44 ` Davide Libenzi
2007-06-07 22:03 ` Ulrich Drepper
2007-06-07 22:40 ` Davide Libenzi
2007-06-08 12:07 ` Theodore Tso
2007-06-08 13:01 ` Alan Cox
2007-06-08 18:11 ` Davide Libenzi
2007-06-08 18:26 ` Alan Cox
2007-06-08 18:43 ` Ulrich Drepper
2007-06-08 18:46 ` Al Viro
2007-06-08 18:56 ` Ulrich Drepper
2007-06-08 19:07 ` Linus Torvalds
2007-06-08 19:21 ` Davide Libenzi
2007-06-09 0:03 ` Linus Torvalds
2007-06-09 0:13 ` Davide Libenzi
2007-06-09 0:36 ` Al Viro
2007-06-09 1:19 ` Ulrich Drepper
2007-06-09 1:41 ` Al Viro
2007-06-09 2:10 ` Ulrich Drepper
2007-06-09 15:15 ` Al Viro
2007-06-09 16:26 ` Ulrich Drepper
2007-06-09 16:54 ` Al Viro
2007-06-09 17:04 ` Davide Libenzi
2007-06-09 17:08 ` Davide Libenzi
2007-06-09 17:08 ` Ulrich Drepper
2007-06-09 17:24 ` Al Viro
2007-06-09 19:27 ` Kyle Moffett
2007-06-09 20:06 ` Al Viro
2007-06-09 20:21 ` Linus Torvalds
2007-06-09 20:31 ` Davide Libenzi
2007-06-09 21:41 ` Matt Mackall
2007-06-09 22:12 ` Davide Libenzi
2007-06-09 20:49 ` Al Viro
2007-06-09 21:55 ` Matt Mackall
2007-06-09 23:33 ` Linus Torvalds
2007-06-10 3:35 ` Davide Libenzi
2007-06-10 3:49 ` Davide Libenzi
2007-06-10 3:19 ` Al Viro
2007-06-10 3:48 ` Linus Torvalds
2007-06-10 4:00 ` Al Viro
2007-06-10 4:03 ` Linus Torvalds
2007-06-10 4:06 ` Al Viro
2007-06-10 4:45 ` dean gaudet
2007-06-10 5:06 ` Linus Torvalds
2007-06-10 5:46 ` Al Viro
2007-06-10 17:23 ` Linus Torvalds
2007-06-10 6:35 ` Kari Hurtta
2007-06-10 15:21 ` Alan Cox
2007-06-10 9:14 ` Eric Dumazet
2007-06-10 15:16 ` Alan Cox
2007-06-10 18:19 ` Linus Torvalds
2007-06-10 2:40 ` Al Viro
2007-06-08 19:34 ` Alan Cox
2007-06-08 19:30 ` Alan Cox
2007-06-08 19:37 ` Davide Libenzi
2007-06-08 19:48 ` Alan Cox
2007-06-08 19:51 ` Davide Libenzi
2007-06-08 21:24 ` Alan Cox
2007-06-08 21:59 ` Davide Libenzi
2007-06-08 22:28 ` Alan Cox
2007-06-08 22:38 ` Davide Libenzi
2007-06-11 8:24 ` Xavier Bestel
2007-06-08 19:22 ` Davide Libenzi
2007-06-09 5:41 ` Paul Mackerras
2007-06-09 14:38 ` Kyle Moffett
2007-06-10 6:48 ` Paul Mackerras
2007-06-10 15:56 ` Davide Libenzi
2007-06-10 19:16 ` Davide Libenzi
2007-06-09 17:00 ` Davide Libenzi
2007-06-10 6:26 ` Paul Mackerras
2007-06-10 7:10 ` William Lee Irwin III
2007-06-10 15:52 ` Davide Libenzi
2007-06-08 18:07 ` Davide Libenzi
2007-06-08 18:35 ` Linus Torvalds
2007-06-07 21:57 ` Davide Libenzi
2007-06-08 4:38 ` Eric Dumazet
2007-06-08 5:20 ` Davide Libenzi
2007-06-07 14:25 ` Ulrich Drepper
2007-06-07 17:56 ` Eric Dumazet
2007-06-07 18:03 ` Davide Libenzi
2007-06-07 18:57 ` Eric Dumazet
2007-06-07 18:26 ` Ulrich Drepper
2007-06-07 18:39 ` Davide Libenzi
2007-06-07 18:56 ` Ulrich Drepper
2007-06-07 19:12 ` Davide Libenzi
2007-06-07 20:03 ` Andrew Morton
2007-06-08 2:55 ` Ulrich Drepper
2007-06-08 5:16 ` Davide Libenzi
2007-06-06 23:29 ` Davide Libenzi
2007-06-07 10:06 ` Alan Cox
2007-06-07 10:45 ` Eric Dumazet
2007-06-07 11:27 ` Alan Cox
2007-06-07 15:41 ` Davide Libenzi
2007-06-07 20:10 ` Linus Torvalds
2007-06-07 20:47 ` Eric Dumazet
2007-06-07 21:08 ` Linus Torvalds
2007-06-07 21:41 ` Davide Libenzi
2007-06-07 20:59 ` Guillaume Chazarain
2007-06-07 21:06 ` Guillaume Chazarain
2007-06-07 21:31 ` Ulrich Drepper
2007-06-07 22:22 ` Davide Libenzi
2007-06-07 23:42 ` Linus Torvalds
2007-06-08 0:04 ` Davide Libenzi
2007-06-08 0:59 ` Matt Mackall
2007-06-08 2:25 ` Linus Torvalds
2007-06-08 15:56 ` Jeff Dike
2007-06-07 0:29 ` Arnd Bergmann
2007-06-07 0:33 ` Davide Libenzi
-- strict thread matches above, loose matches on Subject: below --
2007-06-06 22:30 [patch 1/8] fdmap v2 - fdmap core Davide Libenzi
2007-06-07 6:54 ` Eric Dumazet
2007-06-07 7:10 ` Davide Libenzi
2007-06-07 10:39 ` [patch 7/8] fdmap v2 - implement sys_socket2 Eric Dumazet
2007-06-07 15:42 ` Davide Libenzi
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=466864F8.2050903@cosmosbay.com \
--to=dada1@cosmosbay.com \
--cc=akpm@linux-foundation.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=davidel@xmailserver.org \
--cc=drepper@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=mrmacman_g4@mac.com \
--cc=torvalds@linux-foundation.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox