[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate
From: |
Florian Weimer |
Subject: |
Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate |
Date: |
Mon, 05 Jun 2023 11:28:33 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
* Sergey Bugaev:
> * getpt, openpty: Opening an unused pty, which can't be our ctty
Please add this as a comment to the open operation.
> * shm_open, sem_open: These don't work with ttys
> * opendir: Directories are unlikely to be ttys
I scrolled through the patch, and it seems to me that all these open
calls could use O_NOCTTY. Do you still need a flag because the
performance optimization is not about changing the controlling terminal,
but about detecting the controlling terminal as such?
Thanks,
Florian
- [PATCH v3 0/2] O_IGNORE_CTTY everywhere, Sergey Bugaev, 2023/06/04
- [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Sergey Bugaev, 2023/06/04
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate,
Florian Weimer <=
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Paul Eggert, 2023/06/05
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Sergey Bugaev, 2023/06/06
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Paul Eggert, 2023/06/09
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Sergey Bugaev, 2023/06/09
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Paul Eggert, 2023/06/09
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Sergey Bugaev, 2023/06/10
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Paul Eggert, 2023/06/11
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Sergey Bugaev, 2023/06/13
- Re: [PATCH v3 2/2] Use O_IGNORE_CTTY where appropriate, Paul Eggert, 2023/06/14