On Mon, Oct 23, 2023 at 02:28:04PM -0600, Dan Raymond wrote: > > Anyway, it's up to the x86 maintainers now, good luck! > > > > But personally, I don't see the real need for this at all. It's a > > debugging thing for what exactly? Who needs this? Who will use it? > > When will they use it? And why? > > This comment confuses me. As you know I originally submitted a patch > that added I/O tracing just to the 8250 serial driver. The patch was > titled "create debugfs interface for UART register tracing". You said > this at the time: > > "Anyway, again, cool feature, I like it, but if you can tie it into > the existing trace framework better (either by using that entirely > which might be best), or at the least, putting your hook into the > data path with it, that would be best." Remember some of us, like myself, get on average 1000+ emails a day that they need to file/delete/review, so what I wrote yesterday I usually can't remember, let alone weeks ago :) > My original patch went through a few revisions before Andy Shevchenko > suggested I should add portio tracing instead in a manner similar to > how CONFIG_TRACE_MMIO_ACCESS works. You agreed. Hence I created this > patch. That's great, but it turns out that the x86 maintainers don't like this, so perhaps that's not going to work out well. I still think the original idea of using tracepoints for serial data would be best, but hey, I don't need this feature :) thanks, greg k-h