[review] Add a NEWS entry for multithreaded symbol loading

[review] Add a NEWS entry for multithreaded symbol loading

[Christian Biesinger (Code Review)]

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/725
......................................................................

Add a NEWS entry for multithreaded symbol loading

Just to let people know that this is available and how to use it.

Also updates the description of the setting to say the default is 0.

gdb/ChangeLog:

2019-11-26  Christian Biesinger  <cbiesinger@google.com>

	* NEWS: Mention the new multithreaded symbol loading.

Change-Id: I263add6aae03b523f0870ad4d1e972eada4b382a
---
M gdb/NEWS
1 file changed, 5 insertions(+), 1 deletion(-)



diff --git a/gdb/NEWS b/gdb/NEWS
index 5fd0f41..0fa71d5 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -58,6 +58,10 @@
  with extensions .gdb, .py and .scm are supported as long as GDB was
  compiled with support for that language.
 
+* GDB now supports multithreaded symbol loading for higher performance.
+  This feature is still in testing, so it is disabled by default.  You
+  can turn it on using 'maint set worker-threads unlimited'.
+
 * Python API
 
   ** The gdb.Value type has a new method 'format_string' which returns a
@@ -152,7 +156,7 @@
 maint set worker-threads
 maint show worker-threads
   Control the number of worker threads that can be used by GDB.  The
-  default is "unlimited", which lets GDB choose a number that is
+  default is 0.  "unlimited" lets GDB choose a number that is
   reasonable.  Currently worker threads are only used when demangling
   the names of linker symbols.
 

-- 
Gerrit-Project: binutils-gdb
Gerrit-Branch: master
Gerrit-Change-Id: I263add6aae03b523f0870ad4d1e972eada4b382a
Gerrit-Change-Number: 725
Gerrit-PatchSet: 1
Gerrit-Owner: Christian Biesinger <cbiesinger@google.com>
Gerrit-MessageType: newchange

Re: [review] Add a NEWS entry for multithreaded symbol loading

[Eli Zaretskii]

> Date: Tue, 26 Nov 2019 17:01:20 -0500
> From: "Christian Biesinger (Code Review)" <gerrit@gnutoolchain-gerrit.osci.io>
> Cc: Christian Biesinger <cbiesinger@google.com>
> 
> Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/725
> ......................................................................
> 
> Add a NEWS entry for multithreaded symbol loading
> 
> Just to let people know that this is available and how to use it.
> 
> Also updates the description of the setting to say the default is 0.
> 
> gdb/ChangeLog:
> 
> 2019-11-26  Christian Biesinger  <cbiesinger@google.com>
> 
> 	* NEWS: Mention the new multithreaded symbol loading.

OK, thanks.

[review] Add a NEWS entry for multithreaded symbol loading

[Tom Tromey (Code Review)]

Tom Tromey has posted comments on this change.

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/725
......................................................................


Patch Set 1: Code-Review+1

Looks reasonable to me, but Eli should review.


-- 
Gerrit-Project: binutils-gdb
Gerrit-Branch: master
Gerrit-Change-Id: I263add6aae03b523f0870ad4d1e972eada4b382a
Gerrit-Change-Number: 725
Gerrit-PatchSet: 1
Gerrit-Owner: Christian Biesinger <cbiesinger@google.com>
Gerrit-Reviewer: Tom Tromey <tromey@sourceware.org>
Gerrit-Comment-Date: Wed, 27 Nov 2019 18:03:13 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Re: [review] Add a NEWS entry for multithreaded symbol loading

[Eli Zaretskii]

> Date: Wed, 27 Nov 2019 13:03:13 -0500
> From: "Tom Tromey (Code Review)" <gerrit@gnutoolchain-gerrit.osci.io>
> 
> Tom Tromey has posted comments on this change.
> 
> Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/725
> ......................................................................
> 
> 
> Patch Set 1: Code-Review+1
> 
> Looks reasonable to me, but Eli should review.

I already did.

[pushed] Add a NEWS entry for multithreaded symbol loading

[Sourceware to Gerrit sync (Code Review)]

Sourceware to Gerrit sync has submitted this change.

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/725
......................................................................

Add a NEWS entry for multithreaded symbol loading

Just to let people know that this is available and how to use it.

Also updates the description of the setting to say the default is 0.

gdb/ChangeLog:

2019-11-26  Christian Biesinger  <cbiesinger@google.com>

	* NEWS: Mention the new multithreaded symbol loading.

Change-Id: I263add6aae03b523f0870ad4d1e972eada4b382a
---
M gdb/NEWS
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Tom Tromey: Looks good to me, but someone else must approve


diff --git a/gdb/NEWS b/gdb/NEWS
index cf8c41c..d901407 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -58,6 +58,10 @@
  with extensions .gdb, .py and .scm are supported as long as GDB was
  compiled with support for that language.
 
+* GDB now supports multithreaded symbol loading for higher performance.
+  This feature is still in testing, so it is disabled by default.  You
+  can turn it on using 'maint set worker-threads unlimited'.
+
 * Python API
 
   ** The gdb.Value type has a new method 'format_string' which returns a
@@ -152,7 +156,7 @@
 maint set worker-threads
 maint show worker-threads
   Control the number of worker threads that can be used by GDB.  The
-  default is "unlimited", which lets GDB choose a number that is
+  default is 0.  "unlimited" lets GDB choose a number that is
   reasonable.  Currently worker threads are only used when demangling
   the names of linker symbols.
 

-- 
Gerrit-Project: binutils-gdb
Gerrit-Branch: master
Gerrit-Change-Id: I263add6aae03b523f0870ad4d1e972eada4b382a
Gerrit-Change-Number: 725
Gerrit-PatchSet: 2
Gerrit-Owner: Christian Biesinger <cbiesinger@google.com>
Gerrit-Reviewer: Tom Tromey <tromey@sourceware.org>
Gerrit-MessageType: merged