Re: Kernel Debugging Support
From: Sarah Sharp
Date: Mon Aug 04 2014 - 12:46:47 EST
On Sat, Aug 02, 2014 at 12:47:59AM -0400, Nick Krause wrote:
> Hey Sharp,
Hi Krause,
Please Cc the new xHCI driver maintainer, Mathias Nyman. I'm officially
retired from USB development and have moved onto other projects. :)
> After reading around seems people want support for usb debugging in
> kgdb or other usb based solutions.
Yes, early boot USB debug over xHCI has been a low-priority feature
request.
Unfortunately, USB debug is an optional feature for xHCI host
controllers, so many of them don't have USB debug ports. Even if it is
included, the USB debug ports are often either routed to internal USB
ports or not exposed on the board altogether. Since it's not widely
available, it's not a high priority to implement.
> If you and the other developers are able to help me out a bit as I am
> new I can definitively write this
> area of kgdb support.
The kgdb support is all there. It's the xHCI host controller driver
debug port support that is needed.
Hmm, I only see one commit from your email address in Greg's tree. I
think you should work on some smaller clean ups and bug fixes, and get
some more patches into mainline before you go on to tackle a larger
feature like this. Perhaps Mathias has a smaller task that would be
good for you to tackle?
There's a good tutorial here on how to create a kernel patch, and
respond properly to patch review and rework requests:
http://kernelnewbies.org/OPWfirstpatch
And here's my personal philosophy on how to create a patchset:
http://sarah.thesharps.us/2013/05/08/patchsets-for-dinner/
> Regards Nick
> P.S. If you want Sharp I can change the commit message on my other
> commit you didn't like if me or you
> talk to Greg in order to remove in from mainline, if no that's OK too.
I don't understand what the above sentence means. What commit message
are you referencing? What do you mean by "remove in from mainline"?
Try again?
Sarah Sharp
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/