augusto2112 added a comment. > So it might be nice to add support for vAttachOrWait, along with the query > packet, in this patch if you have the time?
Hi @clayborg, I saw that that implementing vAttachOrWait was already 90% done, so I did just that :) The current patch as-is supports vAttachOrWait. My question was more on the lines of if keeping the query for vAttachOrWait made sense not that it's implemented on Linux. Are there be other targets where it isn't implemented? I also left a comment with a couple of questions earlier, but I think it got hidden by my last comment. I'll repost it here: > @clayborg I've added support for the 'waitfor-interval' and > 'waitfor-duration' flags. Yesterday I thought they existed in macOS, but now > I'm not so sure, as I couldn't find them on "Options.td". They were added in > 2009, so maybe they did exist but were dropped later. Or I just didn't look > at the right place. > > A couple of questions: > > - About the default polling interval, I've set it to 1ms for now, but I found > that (in my system) this keeps a core at 100%. 10ms keeps still keeps it at > around 80%, and 100ms keeps it at around 35%. Maybe 100ms is a good balance? > - On "Options.td", besides the "process attach" command, there's also a > "platform process attach". I haven't touched it since I'm not familiar with > the platform command. Should I add these flags to the platform counterpart as > well? Could you confirm for me that 'waitfor-interval' and 'waitfor-duration' don't exist on macOS? If I'm sending a different message format than what's done on macOS that'd be an issue. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93895/new/ https://reviews.llvm.org/D93895 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
