Re: [RFC][PATCH] Expanding the size of "start" and "end" field in "struct resource"

Vivek Goyal <[email protected]> writes:

> Few problems which I have noticed so far.
>
> - Many printk() warnings. Wherever start and end are being printed,
>   the format specifier being used is %lx. Needs to be changed to %Lx.

Sane, but we need to check the 64bit case as well.

> - Some folks save a pointer of type (unsigned long *) to start and end field
>   and then try to operate on it. This pointer type shall have to be changed
>   to something like u64*.
>
> 	unsigned long *port, *end, *tport, *tend;
> 	port = &dev->res.port_resource[idx].start;

Weird.

> - Some folks cast "start" to a pointer and then use it. Compiler gives warning.
>
> 	addr_reg = (void __iomem *) addr->start;

I'm not familiar enough with that part of the code off the top of my head
but that except for a few helper functions that kind of behavior should
be pretty much forbidden.

This feels like entering the guts of ugly barely working drivers at
the moment.

Eric

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

---

• References:
  • [RFC][PATCH] Expanding the size of "start" and "end" field in "struct resource"
    • From: Vivek Goyal <[email protected]>
  • Re: [RFC][PATCH] Expanding the size of "start" and "end" field in "struct resource"
    • From: Arjan van de Ven <[email protected]>
  • Re: [RFC][PATCH] Expanding the size of "start" and "end" field in "struct resource"
    • From: Kumar Gala <[email protected]>
  • Re: [RFC][PATCH] Expanding the size of "start" and "end" field in "struct resource"
    • From: [email protected] (Eric W. Biederman)
  • Re: [RFC][PATCH] Expanding the size of "start" and "end" field in "struct resource"
    • From: Vivek Goyal <[email protected]>

• Prev by Date: Re: does swsusp suck aftre resume for you? [was Re: Re: Faster resuming of suspend technology.]
• Next by Date: Re: [RFC][PATCH] Expanding the size of "start" and "end" field in "struct resource"
• Previous by thread: Re: [RFC][PATCH] Expanding the size of "start" and "end" field in "struct resource"
• Next by thread: Re: [RFC][PATCH] Expanding the size of "start" and "end" field in "struct resource"
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]