From: Denis Medvedev <nbr@altlinux.org>
To: Alexey Sheplyakov <asheplyakov@basealt.ru>
Cc: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] Q: CONFIG_PSI_DEFAULT_DISABLED=y
Date: Wed, 8 Sep 2021 15:58:41 +0300
Message-ID: <20210908155841.06ab8f74@homerun.localdomain> (raw)
In-Reply-To: <717cfd00-8805-7f20-6ca0-3de8b92f89a1@basealt.ru>
В Wed, 8 Sep 2021 14:47:35 +0400
Alexey Sheplyakov <asheplyakov@basealt.ru> пишет:
> On 08.09.2021 14:14, Denis Medvedev wrote:
> > В Wed, 8 Sep 2021 13:17:06 +0400
>
> >>> Вот мне бы интересно было, если бы malloc мог (при включении
> >>> какой-либо опции) при отсутствии памяти ждать ее появления ,
> >>> выдавая процессу SIGTSTP.
> >>
> >> Сколько примерно времени займёт адаптация userspace к новому
> >> поведению? В частности, как определить те места, где ответ "сейчас
> >> нет, жду" заведомо неприемлем?
> > "сейчас нет, жду" - это не ответ, не код возврата, это поведение.
>
> Тогда это ещё хуже.
>
> > А почему вообще userspace это интересно?
>
> 1. Потому что время имеет значение.
Когда как. Поэтому такое поведение стоит включать явно каким-либо
специальным ключом компиляции, там где такое поведение не будет
фатальным.
> 2. Приход сигнала, в том месте, где его не ожидает - это очень, очень
> плохо.
Невыделение ожидаемой памяти обычно плохо не меньше.
>
> > Ему malloc возвратит память - чаще чем обычно. И всё. Оно просто
> > чаще не получит NULL, его либо снимут явным kill ВНУТРИ malloc либо
> > дадут памяти и волшебный пендель SIGCONT. Зато это явно безопаснее,
> > чем если далее не будет проверен код возврата и будет разыменован
> > NULL.
>
> С точностью до наоборот: появляется ещё одно место, где возможен
> нелокальный выход, и дополнительные возможности для атаки (я даже не
> говорю про DoS).
Нелокальный выход возможен всегда по получении SIGKILL.
>
> > Привеите пример ситуации, когда ответ "сейчас нет, жду" хуже чем
> > "нет памяти"?
>
> С ходу:
>
> pthread_mutex_lock(&mutex);
> result = realloc(global_buf, size*2); /* и тут закончилась память */
Ну и что? Ну подождет realloc выделения памяти в системе и продолжит.
Что тут такого? Проблема будет если кто-то стопнутый процесс или
нить прибьет
- ну а никто не мешает его и просто так прибить SIGKILL-ом.
pthread_mutex_lock ведь касается только набора нитей, а следящий
процесс к этим нитям отношения не имеет и ему эти локи до лампочки.
Он отдельный процесс Linux если что.
>
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel
next prev parent reply other threads:[~2021-09-08 12:58 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-06 12:41 Alexey Shabalin
2021-09-06 13:03 ` Alexey V. Vissarionov
2021-09-06 13:04 ` Andrey Savchenko
2021-09-06 14:32 ` Alexey Sheplyakov
2021-09-06 15:03 ` Andrey Savchenko
2021-09-06 15:23 ` Alexey Sheplyakov
2021-09-06 15:47 ` Andrey Savchenko
2021-09-07 9:16 ` Alexey Sheplyakov
2021-09-06 16:11 ` Sergey Y. Afonin
2021-09-07 7:46 ` Alexey Sheplyakov
2021-09-07 9:32 ` Anton Farygin
2021-09-07 10:17 ` Sergey Afonin
2021-09-07 10:21 ` Alexey Sheplyakov
2021-09-07 10:26 ` Sergey Afonin
2021-09-07 10:59 ` Anton Farygin
2021-09-07 15:15 ` Andrey Savchenko
2021-09-08 7:23 ` Alexey Sheplyakov
2021-09-07 10:58 ` Anton Farygin
2021-09-07 20:47 ` Sergey Y. Afonin
2021-09-08 5:33 ` Anton Farygin
2021-09-08 8:36 ` Sergey Afonin
2021-09-08 8:39 ` Anton Farygin
2021-09-08 8:50 ` Alexey Sheplyakov
2021-09-08 9:11 ` Denis Medvedev
2021-09-08 9:12 ` Anton Farygin
2021-09-08 10:17 ` Denis Medvedev
2021-09-08 10:27 ` Alexey Sheplyakov
2021-09-08 11:09 ` Denis Medvedev
2021-09-08 13:19 ` Alexey Sheplyakov
2021-09-08 10:44 ` Oleg Solovyov
2021-09-08 9:17 ` Alexey Sheplyakov
2021-09-08 10:14 ` Denis Medvedev
2021-09-08 10:47 ` Alexey Sheplyakov
2021-09-08 12:58 ` Denis Medvedev [this message]
2021-09-08 7:08 ` Alexey Sheplyakov
2021-09-08 8:38 ` Sergey Afonin
2021-09-08 8:42 ` Anton Farygin
2021-09-08 9:01 ` Alexey Sheplyakov
2021-09-08 9:40 ` Sergey Afonin
2021-09-08 10:24 ` Anton Farygin
2021-09-08 14:12 ` Alexey Sheplyakov
2021-09-08 14:49 ` Anton Farygin
2021-09-08 15:06 ` Sergey Y. Afonin
2021-09-08 15:37 ` Alexey Sheplyakov
2021-09-08 16:52 ` Sergey Y. Afonin
2021-09-08 10:42 ` Andrey Savchenko
2021-09-07 10:16 ` Sergey Afonin
2021-09-07 10:27 ` Alexey Sheplyakov
2021-09-07 10:31 ` Sergey Afonin
2021-09-07 11:01 ` Anton Farygin
2021-09-07 11:31 ` Alexey Sheplyakov
2021-09-07 11:40 ` Anton Farygin
2021-09-06 15:16 ` Anton V. Boyarshinov
2021-09-06 16:57 ` Vitaly Chikunov
2021-09-06 18:28 ` Anton V. Boyarshinov
2021-09-06 20:49 ` Alexey V. Vissarionov
2021-09-07 9:31 ` Alexey Sheplyakov
2021-09-07 10:26 ` Alexey V. Vissarionov
2021-09-07 10:42 ` Alexey Sheplyakov
2021-09-07 10:56 ` Sergey Afonin
2021-09-07 15:02 ` Andrey Savchenko
2021-09-07 11:21 ` Alexey V. Vissarionov
2021-09-07 11:41 ` Alexey Sheplyakov
2021-09-07 11:44 ` Anton Farygin
2021-09-07 20:54 ` Sergey Y. Afonin
2021-09-07 10:59 ` [devel] [JT] " Oleg Solovyov
2021-09-07 13:59 ` Leonid Krivoshein
2021-09-07 10:54 ` [devel] " Leonid Krivoshein
2021-09-07 11:38 ` Alexey V. Vissarionov
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=20210908155841.06ab8f74@homerun.localdomain \
--to=nbr@altlinux.org \
--cc=asheplyakov@basealt.ru \
--cc=devel@lists.altlinux.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
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git