ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Alexey Sheplyakov <asheplyakov@basealt.ru>
To: Denis Medvedev <nbr@altlinux.org>
Cc: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] Q: CONFIG_PSI_DEFAULT_DISABLED=y
Date: Wed, 8 Sep 2021 14:47:35 +0400
Message-ID: <717cfd00-8805-7f20-6ca0-3de8b92f89a1@basealt.ru> (raw)
In-Reply-To: <20210908131449.500bc86b@homerun.localdomain>

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).

> Привеите пример ситуации, когда ответ "сейчас нет, жду" хуже чем 
> "нет памяти"?

С ходу:

pthread_mutex_lock(&mutex);
result = realloc(global_buf, size*2); /* и тут закончилась память */




  reply	other threads:[~2021-09-08 10:47 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 [this message]
2021-09-08 12:58                               ` Denis Medvedev
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=717cfd00-8805-7f20-6ca0-3de8b92f89a1@basealt.ru \
    --to=asheplyakov@basealt.ru \
    --cc=devel@lists.altlinux.org \
    --cc=nbr@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