On Wed, 2007-07-18 at 14:45 -0700, Luck, Tony wrote:
> > The type compat_u64 on ia64 is defined if CONFIG_COMPAT is defined
> > (which is rare these days on ia64) So, it appears we are missing
> > an #ifdef CONFIG_COMPAT somewhere in fs/quota.c. I will leave that
> > up to the original author to determine where.
>
> Hmmmm ... all of my test config files have CONFIG_COMPAT=y, so I didn't
> notice this. I'll have to fix one of them to turn it off.
>
> Perhaps this (untested) is what is needed?
>
> diff --git a/fs/quota.c b/fs/quota.c
> index e6577ac..ac3af87 100644
> --- a/fs/quota.c
> +++ b/fs/quota.c
> @@ -387,7 +387,7 @@ asmlinkage long sys_quotactl(unsigned int cmd, const char __user *special, qid_t
> return ret;
> }
>
> -#if defined(CONFIG_X86_64) || defined(CONFIG_IA64)
> +#if defined(CONFIG_X86_64) || (defined(CONFIG_IA64) && defined(CONFIG_COMPAT))
> /*
> * This code works only for 32 bit quota tools over 64 bit OS (x86_64, ia64)
> * and is necessary due to alignment problems.
Works for me, compiles just fine now.
ACK
- Doug
-
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/
[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]
