Am 11/07/2022 um 15:08 schrieb Vladimir Sementsov-Ogievskiy: > > That made me ask: > > 1. Are all tests always run in main loop? If yes, why to protect status > reading in test_complete_in_standby() ? > > 2. Maybe, we don't need to protect anything here? Why to protect other > things if we run everything in main loop? I think it's still good example and practice to protect a function if it needs to be protected and its name ends with _locked. It would just confuse the reader if we don't protect it. Emanuele
- Re: [PATCH v9 19/21] blockjo... Emanuele Giuseppe Esposito
- [PATCH v9 21/21] job: remove unused functions Emanuele Giuseppe Esposito
- [PATCH v9 20/21] blockjob: remove unused func... Emanuele Giuseppe Esposito
- [PATCH v9 15/21] job.c: enable job lock/unloc... Emanuele Giuseppe Esposito
- Re: [PATCH v9 15/21] job.c: enable job l... Vladimir Sementsov-Ogievskiy
- Re: [PATCH v9 15/21] job.c: enable j... Emanuele Giuseppe Esposito
- [PATCH v9 13/21] job: detect change of aiocon... Emanuele Giuseppe Esposito
- Re: [PATCH v9 13/21] job: detect change ... Vladimir Sementsov-Ogievskiy
- [PATCH v9 09/21] jobs: use job locks also in ... Emanuele Giuseppe Esposito
- Re: [PATCH v9 09/21] jobs: use job locks... Vladimir Sementsov-Ogievskiy
- Re: [PATCH v9 09/21] jobs: use job l... Emanuele Giuseppe Esposito
- Re: [PATCH v9 09/21] jobs: use j... Vladimir Sementsov-Ogievskiy
- Re: [PATCH v9 09/21] jobs: u... Emanuele Giuseppe Esposito
- [PATCH v9 16/21] block_job_query: remove atom... Emanuele Giuseppe Esposito
- Re: [PATCH v9 16/21] block_job_query: re... Vladimir Sementsov-Ogievskiy
- Re: [PATCH v9 00/21] job: replace AioContext ... Vladimir Sementsov-Ogievskiy
- Re: [PATCH v9 00/21] job: replace AioCon... Emanuele Giuseppe Esposito