[PATCH][Jaunty SRU][Intrepid SRU] UBUNTU: Fix the error value in /proc/iomem passed by the kernel paramter "reserve=" on X86_64
Keng-Yü Lin
keng-yu.lin at canonical.com
Tue Jun 30 07:21:05 UTC 2009
Sorry I have a silly typo, please s/gif/git/
git://kernel.ubuntu.com/lexical/lexical-karmic.git lp370003
git://kernel.ubuntu.com/lexical/lexical-jaunty.git lp370003
git://kernel.ubuntu.com/lexical/lexical-intrepid.git lp370003
2009/6/30 <keng-yu.lin at canonical.com>:
> From: Keng-Yu Lin <keng-yu.lin at canonical.com>
>
> SRU Justificatoin:
>
> IMPACT: kernel parameter "reserve=" is not handled well on AMD64 and the value in /proc/iomem is not padded by 0xf in the front. The bug LP#187671 depends on this bug.
>
> FIX: A patch from Zhang Rui fixed. The patch is also summitted to the mainline. The status on mainline bugzilla is "RESOLVED CODE_FIX". It may be merged into the mainline some day.
>
> TEST: I tested on Karmic, Jaunty and Intrepid. The value in /proc/iomem is correct after applying this patch. The original reporter also tested on 2.6.28-11 kernel and reported to work well.
>
> OriginalAuthor: Zhang Rui (rui.zhang at intel.com)
> OriginalLocation: http://bugzilla.kernel.org/show_bug.cgi?id=13253
> BugLink: https://bugs.launchpad.net/bugs/370003
> Bug: #370003
>
> Request-pull
> git://kernel.ubuntu.com/lexical/lexical-karmic.gif lp370003
> git://kernel.ubuntu.com/lexical/lexical-jaunty.gif lp370003
> git://kernel.ubuntu.com/lexical/lexical-intrepid.gif lp370003
>
> They are basically the same; the line numbers differ.
>
> Signed-off-by: Keng-Yu Lin <keng-yu.lin at canonical.com>
> ---
> kernel/resource.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/kernel/resource.c b/kernel/resource.c
> index ac5f3a3..78b0872 100644
> --- a/kernel/resource.c
> +++ b/kernel/resource.c
> @@ -787,7 +787,7 @@ static int __init reserve_setup(char *str)
> static struct resource reserve[MAXRESERVE];
>
> for (;;) {
> - int io_start, io_num;
> + unsigned int io_start, io_num;
> int x = reserved;
>
> if (get_option (&str, &io_start) != 2)
> --
> 1.6.0.4
>
>
> --
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>
More information about the kernel-team
mailing list