Hi,

The vAttach packet, which is used to implement the attach command
in the extended-remote target, in it's documentation says that if the
stub is currently controlling a process, it is killed.

`vAttach;pid'
    Attach to a new process with the specified process ID. pid is a 
hexadecimal integer identifying the process. If the stub is currently 
controlling a process, it is killed. The attached process is stopped.

This requirement isn't really needed, since gdb already takes care
of killing the running inferior.

We would like to remove this requirement, since it has no real value,
and limits future extensions for stubs controlling more than one
process at a time.

So, that's what the attached patch does.  It removes that frase,
and removes the implementation for it in the probably only
current implementation of it: gdbserver on HEAD.

I'd like to have this committed before the release, since there's
no release with this packet yet.

Tested with a local gdbserver on x86_64-unknown-linux-gnu, no
regressions, and by looking at the generated html docs.

-- 
Pedro Alves