diff mbox series

[RFC,v1,2/2] Documentation: btrfs: Document the influence of wq_cpu_set to thread_pool option

Message ID 20230226162639.20559-3-ammarfaizi2@gnuweeb.org (mailing list archive)
State New, archived
Headers show
Series Documentation: Introducing `wq_cpu_set` mount option for btrfs | expand

Commit Message

Ammar Faizi Feb. 26, 2023, 4:26 p.m. UTC
If wq_cpu_set option is set, the default thread_pool value will be
adjusted accordingly.

Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
---
 Documentation/ch-mount-options.rst | 3 +++
 1 file changed, 3 insertions(+)

Comments

Bagas Sanjaya Feb. 27, 2023, 2:15 a.m. UTC | #1
On Sun, Feb 26, 2023 at 11:26:39PM +0700, Ammar Faizi wrote:
> +        Since 6.5, if *wq_cpu_set* is set, the default value will be the number of
> +        online CPUs in the CPU wq_cpu_set plus 2.
> +
  
Why will the behavior be introduced in such future version (6.5)?
Ammar Faizi Feb. 27, 2023, 10:02 a.m. UTC | #2
On Mon, Feb 27, 2023 at 09:15:58AM +0700, Bagas Sanjaya wrote:
> Why will the behavior be introduced in such future version (6.5)?

It's not like it has been staged for the next merge window. It's still
in an RFC state. The changes are not trivial and need further review.

I don't know if it can hit the next merge window. As such, I picked a
long distance for this proposal. If it ends up going upstream sooner, we
can change this document.
Bagas Sanjaya March 1, 2023, 2:06 a.m. UTC | #3
On 2/27/23 17:02, Ammar Faizi wrote:
> On Mon, Feb 27, 2023 at 09:15:58AM +0700, Bagas Sanjaya wrote:
>> Why will the behavior be introduced in such future version (6.5)?
> 
> It's not like it has been staged for the next merge window. It's still
> in an RFC state. The changes are not trivial and need further review.
> 
> I don't know if it can hit the next merge window. As such, I picked a
> long distance for this proposal. If it ends up going upstream sooner, we
> can change this document.
> 

OK, thanks!
diff mbox series

Patch

diff --git a/Documentation/ch-mount-options.rst b/Documentation/ch-mount-options.rst
index 48fe63ee5e95c297..c38caf5e5fd0b719 100644
--- a/Documentation/ch-mount-options.rst
+++ b/Documentation/ch-mount-options.rst
@@ -411,6 +411,9 @@  thread_pool=<number>
         due to increased locking contention, process scheduling, cache-line bouncing or
         costly data transfers between local CPU memories.
 
+        Since 6.5, if *wq_cpu_set* is set, the default value will be the number of
+        online CPUs in the CPU wq_cpu_set plus 2.
+
 treelog, notreelog
         (default: on)