[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#561891: Bug 561891: Is FTBFS for fio on SuperH (sh4) resolved?



Hi,

2011/8/27 Jens Axboe <axboe@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

Attachment: fio-test.fio
Description: Binary data

From 4d899cd0257b113c7e80f6ed2bd9ab69f59dc998 Mon Sep 17 00:00:00 2001
From: Nobuhiro Iwamatsu <iwamatsu@nigauri.org>
Date: Sun, 28 Aug 2011 06:34:37 +0900
Subject: [PATCH] Fix compile on environment of SuperH

Signed-off-by: Nobuhiro Iwamatsu <iwamatsu@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


Reply to: