Hi, 2011/8/27 Jens Axboe <ax...@kernel.dk>: > On 2011-08-26 15:00, Martin Steigerwald wrote: >> I can take it as patch in the debian package as well, Jens, if you prefer >> not to include it upstream. But I guess you have no problem with including >> it, since you have already done so. Please tell me if otherwise. > > As mentioned, the patch is already included. However, I would really > appreciate if Iwamatsu-san would give it a compile and runtime test so > that I know it's good. I don't have sh cross compilers on my system. And > I'm pretty close to tagging the next release, would be a shame if 1.58 > was released with broken sh support. >
I checked build and test. First, because fio was not able to build, I revised it. I attach a patch. And I tested with tiny config file. Work fine. I attached config file. Best regards, Nobuhiro -- Nobuhiro Iwamatsu iwamatsu at {nigauri.org / debian.org} GPG ID: 40AD1FA6
fio-test.fio
Description: Binary data
From 4d899cd0257b113c7e80f6ed2bd9ab69f59dc998 Mon Sep 17 00:00:00 2001 From: Nobuhiro Iwamatsu <iwama...@nigauri.org> Date: Sun, 28 Aug 2011 06:34:37 +0900 Subject: [PATCH] Fix compile on environment of SuperH Signed-off-by: Nobuhiro Iwamatsu <iwama...@nigauri.org> --- arch/arch-sh.h | 5 +++++ arch/arch.h | 14 +++++++------- 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/arch/arch-sh.h b/arch/arch-sh.h index ef4ee03..f5f313d 100644 --- a/arch/arch-sh.h +++ b/arch/arch-sh.h @@ -33,6 +33,11 @@ #define read_barrier() mb() #define write_barrier() mb() +#include <stdio.h> +#include <elf.h> + +extern unsigned long arch_flags; + #define CPU_HAS_LLSC 0x0040 static inline int arch_init(char *envp[]) diff --git a/arch/arch.h b/arch/arch.h index 16f4c3a..d598652 100644 --- a/arch/arch.h +++ b/arch/arch.h @@ -23,6 +23,13 @@ enum { arch_generic, }; +enum { + ARCH_FLAG_1 = 1 << 0, + ARCH_FLAG_2 = 1 << 1, + ARCH_FLAG_3 = 1 << 2, + ARCH_FLAG_4 = 1 << 3, +}; + #if defined(__i386__) #include "arch-x86.h" #elif defined(__x86_64__) @@ -65,11 +72,4 @@ static inline int arch_init(char *envp[]) } #endif -enum { - ARCH_FLAG_1 = 1 << 0, - ARCH_FLAG_2 = 1 << 1, - ARCH_FLAG_3 = 1 << 2, - ARCH_FLAG_4 = 1 << 3, -}; - #endif -- 1.7.5.4