Re: [PATCH v2 2/2] modules/kmod: replace implementation with a sempahore

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v2 2/2] modules/kmod: replace implementation with a sempahore
From: Luis Chamberlain <mcgrof@xxxxxxxxxx>
Date: Thu, 13 Apr 2023 09:38:26 -0700
Cc: david@xxxxxxxxxx, patches@xxxxxxxxxxxxxxx, linux-modules@xxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, pmladek@xxxxxxxx, petr.pavlu@xxxxxxxx, prarit@xxxxxxxxxx, torvalds@xxxxxxxxxxxxxxxxxxxx, gregkh@xxxxxxxxxxxxxxxxxxx, rafael@xxxxxxxxxx, christophe.leroy@xxxxxxxxxx, tglx@xxxxxxxxxxxxx, peterz@xxxxxxxxxxxxx, song@xxxxxxxxxx, rppt@xxxxxxxxxx, dave@xxxxxxxxxxxx, willy@xxxxxxxxxxxxx, vbabka@xxxxxxx, mhocko@xxxxxxxx, dave.hansen@xxxxxxxxxxxxxxx, colin.i.king@xxxxxxxxx, jim.cromie@xxxxxxxxx, catalin.marinas@xxxxxxx, jbaron@xxxxxxxxxx, rick.p.edgecombe@xxxxxxxxx
In-reply-to: <alpine.LSU.2.21.2304131136580.27633@pobox.suse.cz>
References: <20230405203505.1343562-1-mcgrof@kernel.org> <20230405203505.1343562-3-mcgrof@kernel.org> <alpine.LSU.2.21.2304131136580.27633@pobox.suse.cz>

On Thu, Apr 13, 2023 at 11:44:51AM +0200, Miroslav Benes wrote:
> Hi,

<spell checks>

Thanks, fixed!

> MAX_KMOD_ALL_BUSY_TIMEOUT * HZ ?

Yes! In the end schedule_timeout() is used so yup, thanks!

  Luis

References:
- [PATCH v2 0/2] kmod: simplify with a semaphore
  - From: Luis Chamberlain
- [PATCH v2 2/2] modules/kmod: replace implementation with a sempahore
  - From: Luis Chamberlain
- Re: [PATCH v2 2/2] modules/kmod: replace implementation with a sempahore
  - From: Miroslav Benes

Prev by Date: Re: [PATCH v2 2/2] modules/kmod: replace implementation with a sempahore
Next by Date: Re: [PATCH 10/17] tty: remove MODULE_LICENSE in non-modules
Previous by thread: Re: [PATCH v2 2/2] modules/kmod: replace implementation with a sempahore
Next by thread: [PATCH v2 1/2] Change DEFINE_SEMAPHORE() to take a number argument
Index(es):
- Date
- Thread

[Index of Archives] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI] [Big List of Linux Books]