misc: Adds a warning in case gdb is attached multiple times

Information
Submitter:	Alexandru Dutu
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 11654:46ce6e3222ec
---------------------------
misc: Adds a warning in case gdb is attached multiple times
Instead of scheduling another event, this patch adds a warning in case gdb
is attached multiple times and the first attachement event has not been
processed yet.

Testing Done

Regressions pass.

Issue Summary

Description	From	Last Updated	Status
Could you make this a bit more descriptive before submitting it? Something like "A GDB trap event has already been ...	Andreas Sandberg	Oct. 4, 2016, 9:06 a.m.	Resolved

Ship It!

Testing Done:

Regressions pass.

Wouldn't it make more sense to just disallow multiple GDB clients to be attached at the same time?

Alexandru Dutu Oct. 3, 2016, 9:25 a.m. (Oct. 3, 2016, 9:25 a.m.)

This does not make sense to me as one can run a configuration with multiple, different processes. For example: threadContext A will be executing process 1 and threadContext B will be executing process 2. In this case 2 gdb listeners will be instantiated and each will be listening on a different port. Imho, having multiple GDB clients is a neat feature.

Andreas Sandberg Oct. 3, 2016, 9:47 a.m. (Oct. 3, 2016, 9:47 a.m.)
```
Right, I see what you mean. Just ignore me. :)
```

Doesn't look like "gdb" is one of our approved keywords. Maybe misc? http://gem5.org/Submitting_Contributions#Commit_Messages

Summary:

-	gdb: Adds a warning in case gdb is attached multiple times
+	misc: Adds a warning in case gdb is attached multiple times

Description:

~		Changeset 11654:5cc5f694f938
	~	Changeset 11654:46ce6e3222ec

~		gdb: Adds a warning in case gdb is attached multiple times
	~	misc: Adds a warning in case gdb is attached multiple times
		Instead of scheduling another event, this patch adds a warning in case gdb
		is attached multiple times and the first attachement event has not been
		processed yet.