lkml.org 
[lkml]   [2002]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH] serial.c procfs kudzu 2nd try - discussion
Date
Jeff Garzik wrote:
>
> Russell King wrote:
>
> >On Mon, Mar 11, 2002 at 05:41:09PM -0800, Ed Vance wrote:
> >
> >>2. Does anybody know of anything that will break because of the leading
> >> zeros that are now present on the address field?
> >>
> >
> >I'm not overly happy with this idea - there isn't anything that says an
> >ioport address has 4 digits. I know of machines where an ioport address
> >has 8, and I'm sure on the Alpha or Sparc64 its probably 16 digits.
> >
> Agreed. Standard portability convention seems to say that one treats io
> ports and io mem both as unsigned long, including when printing...

Yeah. It was pretty ugly. My brain was caught in an x86-think well. I will
resubmit with serinfo back to 1.0, all addresses in free form as they were
before, proper lines for memory mapped ports (tagged "port:"), and no lines
generated for nonexistent ports.

Russell King wrote:
>
> Then file a bug against kudzu and get them to fix that so it doesn't
> SEGV when it finds something it doesn't like, and teach it about the
> 'mem' tag.
>
> If kudzu ignores the serinfo: line as well, that's also another kudzu
> bug.

Kudzu ignores all lines that do not contain "uart", including the serinfo
line. I will ask Red Hat to change pciserial.c to look for "mem" if "port"
is not found and handle them interchangeably, so we can use the "mem" tag
later, and to check the returned values of the string functions before
attempting a de-reference.

Thanks for the good input.

Ed

----------------------------------------------------------------
Ed Vance serial24@macrolink.com
Macrolink, Inc. 1500 N. Kellogg Dr Anaheim, CA 92807
----------------------------------------------------------------

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:24    [W:0.044 / U:1.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site