ALT Linux Team development discussions
 help / color / mirror / Atom feed
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


  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