Re: [RFC][PATCH 2/2] x86_64: earlyprintk usb debug device support.

[ebiederm@xmission.com (Eric W. Biederman)]

Greg KH <gregkh@suse.de> writes:

> On Mon, Dec 04, 2006 at 12:18:30PM -0800, Lu, Yinghai wrote:
>> -----Original Message-----
>> From: ebiederm@xmission.com [mailto:ebiederm@xmission.com] 
>> 
>> >arch/x86_64/kernel/early_printk.c |  574
>> +++++++++++++++++++++++++++++++++++++
>> > drivers/usb/host/ehci.h           |    8 +
>> > include/asm-x86_64/fixmap.h       |    1 
>> 
>> Can you separate usbdebug handle out from early_printk? 
>
> Yeah, at least tear it out of x86-64, so those of us stuck on different
> platforms can use this :)
>
> Other than that minor issue, this looks great.  I don't have a x86-64
> box set up here at the moment, so I can't test it, but it looks
> acceptable at first glance.

Makes sense.  I'm curious now what architecture do you have?

Anyway next time I touch this the project will be how to integrate
this into the kernel cleanly.  This round was to figure out how
to get some working code.

If someone beats me to the punch on generalizing this code I won't
mind.

The first pass was a success.  And the performance is reasonable
assuming you don't plug the end you are watching into a usb1 only
port.

Given that I didn't really know anything about usb a week ago I think
I did pretty well :)

Eric