[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <200906091946.10195.arnd@arndb.de>
Date: Tue, 9 Jun 2009 19:46:10 +0200
From: Arnd Bergmann <arnd@...db.de>
To: liqin.chen@...plusct.com
Cc: linux-arch@...r.kernel.org, linux-kernel@...r.kernel.org,
Andrew Morton <akpm@...ux-foundation.org>,
torvalds@...ux-foundation.org
Subject: Re: [PATCH 18/27] score: create head files uaccess.h unaligned.h unistd.h user.h
On Tuesday 09 June 2009, liqin.chen@...plusct.com wrote:
> diff --git a/arch/score/include/asm/unaligned.h
> b/arch/score/include/asm/unaligned.h
> new file mode 100644
> index 0000000..4b8fec0
> --- /dev/null
> +++ b/arch/score/include/asm/unaligned.h
> @@ -0,0 +1,11 @@
> +#ifndef _ASM_SCORE_UNALIGNED_H
> +#define _ASM_SCORE_UNALIGNED_H
> +
> +#include <linux/unaligned/le_struct.h>
> +#include <linux/unaligned/be_byteshift.h>
> +#include <linux/unaligned/generic.h>
> +
> +#define get_unaligned __get_unaligned_le
> +#define put_unaligned __put_unaligned_le
> +
> +#endif /* _ASM_SCORE_UNALIGNED_H */
Any reason not to use the asm-generic version of this file?
If it didn't work, that is probably a bug in my code, not
yours, but they really look compatible.
> diff --git a/arch/score/include/asm/unistd.h
> b/arch/score/include/asm/unistd.h
> new file mode 100644
> index 0000000..0cbfef8
> --- /dev/null
> +++ b/arch/score/include/asm/unistd.h
> @@ -0,0 +1,12 @@
> +#ifndef _ASM_SCORE_UNISTD_H
> +#define _ASM_SCORE_UNISTD_H
> +
> +#define __ARCH_HAVE_MMU
> +#define __ARCH_WANT_SYSCALL_NO_AT
> +#define __ARCH_WANT_SYSCALL_NO_FLAGS
> +#define __ARCH_WANT_SYSCALL_OFF_T
> +#define __ARCH_WANT_SYSCALL_DEPRECATED
> +
> +#include <asm-generic/unistd.h>
I realize that this is the code I added to microblaze in my tree.
However, the four __ARCH_WANT_SYSCALL_* definitions are really
only meant as a transitional helper to get your code ported
from the older ABI. They should be removed from your file in
order to get the short system call list. Have you tried this?
In order to get there, you probably need to add a lot of wrappers
to libc along the lines of
int unlink(const char *pathname)
{
return unlinkat(AT_FDCWD, pathname, 0);
}
Arnd <><
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists