On Tue, 2006-10-17 at 09:51, Jay Lan wrote:
> Jay Lan wrote:
> <snip>
> >
> > Well, i built 2.6.18 kernel and 2.6.18rc5 kernels.
> > I also built 2 kexec-tools based on kdump10 patch. I have been
> > associating the kexec-tools tree that i built at the rc5 time with
> > the rc5 kernel and the newer kexec-tools tree with the 2.6.18
> > kernel.
> >
> > I just ran a few tests with different combination of kexec-tools
> > and kernels. Well, it showed that the rc5 version of kexec-tools
> > worked correctly with both kernels, but the newer version of
> > kexec-tools displayed the system restart problem with both kernels.
> >
> > So, it was the kexec-tools that caused the problem. Not the kernel
> > side. I will diff the patches i applied to both trees...
>
> Nan-hai,
>
> OK, the '-noio' option patch you posted on 9/20 was the cause of
> my system restart.
> http://lists.osdl.org/pipermail/fastboot/2006-September/004062.html
>
> If instead, if i make every inline functions in
> purgatory/arch/ia64/io.h null functions, it would then work.
> I will attach the patch i used (based on your suggestion) here for
> your reference. It was in rough form.
>
>
> Thanks,
> - jay
>
Hi, did you run kexec -p with --noio option?
Zou Nan hai
>
> >
> > Later,
> > - jay
> >
> >
> >
> >> Zou Nan hai
> >>> Thanks,
> >>> - jay
> >> -
> >> To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
> >> the body of a message to majordomo@xxxxxxxxxxxxxxx
> >> More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
> > -
> > To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
> > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
> ______________________________________________________________________
>
> Index: kexec-tools-1.101/purgatory/arch/ia64/io.h
> ===================================================================
> --- kexec-tools-1.101.orig/purgatory/arch/ia64/io.h 2006-10-16 18:19:07.346416494 -0700
> +++ kexec-tools-1.101/purgatory/arch/ia64/io.h 2006-10-16 18:47:14.742913628 -0700
> @@ -4,6 +4,7 @@
> #define MF() asm volatile ("mf.a" ::: "memory")
> #define IO_SPACE_ENCODING(p) ((((p) >> 2) << 12) | (p & 0xfff))
>
> +#ifdef DO_VGA_IO
> static inline void *io_addr (unsigned long port)
> {
> unsigned long offset;
> @@ -91,4 +92,46 @@ static inline void writel(unsigned int b
> {
> *(volatile unsigned int *) addr = b;
> }
> +#else
> +static inline void *io_addr (unsigned long port)
> +{ return (void*)0; }
> +
> +static inline unsigned int inb (unsigned long port)
> +{ return 0; }
> +
> +static inline unsigned int inw (unsigned long port)
> +{ return 0; }
> +
> +static inline unsigned int ia64_inl (unsigned long port)
> +{ return 0; }
> +
> +static inline void outb (unsigned char val, unsigned long port)
> +{ return; }
> +
> +static inline void outw (unsigned short val, unsigned long port)
> +{ return; }
> +
> +static inline void outl (unsigned int val, unsigned long port)
> +{ return; }
> +
> +
> +static inline unsigned char readb(const volatile void *addr)
> +{ return 0; }
> +
> +static inline unsigned short readw(const volatile void *addr)
> +{ return 0; }
> +
> +static inline unsigned int readl(const volatile void *addr)
> +{ return 0; }
> +
> +static inline void writeb(unsigned char b, volatile void *addr)
> +{ return; }
> +
> +static inline void writew(unsigned short b, volatile void *addr)
> +{ return; }
> +
> +static inline void writel(unsigned int b, volatile void *addr)
> +{ return; }
> +
> +#endif
> #endif
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|