Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error if multiprocess not supported #28

Open
tromey opened this issue Dec 22, 2017 · 2 comments
Open

Error if multiprocess not supported #28

tromey opened this issue Dec 22, 2017 · 2 comments

Comments

@tromey
Copy link
Collaborator

tromey commented Dec 22, 2017

I think the current plan is to only support the multiprocess extensions. So, we should error if the client does not send the multiprocess feature in the initial feature query packet.

@tromey
Copy link
Collaborator Author

tromey commented Dec 27, 2017

We can't do this, since lldb doesn't support this extension. I think it's fine to continue parsing multiprocess thread-ids unconditionally, but we should be more careful with the responses we send. Some investigation is needed to see how lldb represents threads - like maybe it sends the TID as the PID?

@tromey
Copy link
Collaborator Author

tromey commented Dec 28, 2017

See https://github.com/tromey/rust-gdb-remote-protocol/tree/client-state

However it's up to the library user to know whether multiple processes are supported, and that branch doesn't yet expose the information. So, a bit more work has to be done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant