linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RFC 5/13] extents and 48bit ext3: sector_t type format string
@ 2006-06-09  1:22 Mingming Cao
  2006-06-09  2:45 ` Andrew Morton
  0 siblings, 1 reply; 2+ messages in thread
From: Mingming Cao @ 2006-06-09  1:22 UTC (permalink / raw)
  To: linux-kernel, ext2-devel, linux-fsdevel



Define SECTOR_FMT to print sector_t in proper format

Signed-off-by: Dave Kleikamp <shaggy@austin.ibm.com>
Acked-by: Andreas Dilger <adilger@clusterfs.com>


---

 linux-2.6.16-ming/include/asm-h8300/types.h   |    1 +
 linux-2.6.16-ming/include/asm-i386/types.h    |    1 +
 linux-2.6.16-ming/include/asm-mips/types.h    |    5 +++++
 linux-2.6.16-ming/include/asm-powerpc/types.h |    5 +++++
 linux-2.6.16-ming/include/asm-s390/types.h    |    5 +++++
 linux-2.6.16-ming/include/asm-sh/types.h      |    1 +
 linux-2.6.16-ming/include/asm-x86_64/types.h  |    1 +
 linux-2.6.16-ming/include/linux/types.h       |    1 +
 8 files changed, 20 insertions(+)

diff -puN include/asm-h8300/types.h~sector_fmt include/asm-h8300/types.h
--- linux-2.6.16/include/asm-h8300/types.h~sector_fmt	2006-06-07 15:43:40.000000000 -0700
+++ linux-2.6.16-ming/include/asm-h8300/types.h	2006-06-07 15:43:40.000000000 -0700
@@ -57,6 +57,7 @@ typedef u32 dma_addr_t;
 
 #define HAVE_SECTOR_T
 typedef u64 sector_t;
+#define SECTOR_FMT "%llu"
 
 #define HAVE_BLKCNT_T
 typedef u64 blkcnt_t;
diff -puN include/asm-i386/types.h~sector_fmt include/asm-i386/types.h
--- linux-2.6.16/include/asm-i386/types.h~sector_fmt	2006-06-07 15:43:40.000000000 -0700
+++ linux-2.6.16-ming/include/asm-i386/types.h	2006-06-07 15:43:40.000000000 -0700
@@ -60,6 +60,7 @@ typedef u64 dma64_addr_t;
 
 #ifdef CONFIG_LBD
 typedef u64 sector_t;
+#define SECTOR_FMT "%llu"
 #define HAVE_SECTOR_T
 #endif
 
diff -puN include/asm-mips/types.h~sector_fmt include/asm-mips/types.h
--- linux-2.6.16/include/asm-mips/types.h~sector_fmt	2006-06-07 15:43:40.000000000 -0700
+++ linux-2.6.16-ming/include/asm-mips/types.h	2006-06-07 15:43:40.000000000 -0700
@@ -96,6 +96,11 @@ typedef unsigned long phys_t;
 
 #ifdef CONFIG_LBD
 typedef u64 sector_t;
+#if (_MIPS_SZLONG == 64)
+#define SECTOR_FMT "%lu"
+#else
+#define SECTOR_FMT "%llu"
+#endif
 #define HAVE_SECTOR_T
 #endif
 
diff -puN include/asm-powerpc/types.h~sector_fmt include/asm-powerpc/types.h
--- linux-2.6.16/include/asm-powerpc/types.h~sector_fmt	2006-06-07 15:43:40.000000000 -0700
+++ linux-2.6.16-ming/include/asm-powerpc/types.h	2006-06-07 15:43:40.000000000 -0700
@@ -100,6 +100,11 @@ typedef struct {
 
 #ifdef CONFIG_LBD
 typedef u64 sector_t;
+#ifdef __powerpc64__
+#define SECTOR_FMT "%lu"
+#else
+#define SECTOR_FMT "%llu"
+#endif
 #define HAVE_SECTOR_T
 #endif
 
diff -puN include/asm-s390/types.h~sector_fmt include/asm-s390/types.h
--- linux-2.6.16/include/asm-s390/types.h~sector_fmt	2006-06-07 15:43:40.000000000 -0700
+++ linux-2.6.16-ming/include/asm-s390/types.h	2006-06-07 15:43:40.000000000 -0700
@@ -90,6 +90,11 @@ typedef union {
 
 #ifdef CONFIG_LBD
 typedef u64 sector_t;
+#ifndef __s390x__
+#define SECTOR_FMT "%llu"
+#else
+#define SECTOR_FMT "%lu"
+#endif
 #define HAVE_SECTOR_T
 #endif
 
diff -puN include/asm-sh/types.h~sector_fmt include/asm-sh/types.h
--- linux-2.6.16/include/asm-sh/types.h~sector_fmt	2006-06-07 15:43:40.000000000 -0700
+++ linux-2.6.16-ming/include/asm-sh/types.h	2006-06-07 15:43:40.000000000 -0700
@@ -55,6 +55,7 @@ typedef u32 dma_addr_t;
 
 #ifdef CONFIG_LBD
 typedef u64 sector_t;
+#define SECTOR_FMT "%llu"
 #define HAVE_SECTOR_T
 #endif
 
diff -puN include/asm-x86_64/types.h~sector_fmt include/asm-x86_64/types.h
--- linux-2.6.16/include/asm-x86_64/types.h~sector_fmt	2006-06-07 15:43:40.000000000 -0700
+++ linux-2.6.16-ming/include/asm-x86_64/types.h	2006-06-07 15:43:40.000000000 -0700
@@ -49,6 +49,7 @@ typedef u64 dma64_addr_t;
 typedef u64 dma_addr_t;
 
 typedef u64 sector_t;
+#define SECTOR_FMT "%llu"
 #define HAVE_SECTOR_T
 
 #endif /* __ASSEMBLY__ */
diff -puN include/linux/types.h~sector_fmt include/linux/types.h
--- linux-2.6.16/include/linux/types.h~sector_fmt	2006-06-07 15:43:40.000000000 -0700
+++ linux-2.6.16-ming/include/linux/types.h	2006-06-07 15:43:40.000000000 -0700
@@ -135,6 +135,7 @@ typedef		__s64		int64_t;
  */
 #ifndef HAVE_SECTOR_T
 typedef unsigned long sector_t;
+#define SECTOR_FMT "%lu"
 #endif
 
 #ifndef HAVE_BLKCNT_T

_

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [RFC 5/13] extents and 48bit ext3: sector_t type format string
  2006-06-09  1:22 [RFC 5/13] extents and 48bit ext3: sector_t type format string Mingming Cao
@ 2006-06-09  2:45 ` Andrew Morton
  0 siblings, 0 replies; 2+ messages in thread
From: Andrew Morton @ 2006-06-09  2:45 UTC (permalink / raw)
  To: cmm; +Cc: linux-fsdevel, ext2-devel, linux-kernel

On Thu, 08 Jun 2006 18:22:09 -0700
Mingming Cao <cmm@us.ibm.com> wrote:

> 
> 
> Define SECTOR_FMT to print sector_t in proper format
> 
> ...
>
>  #define HAVE_SECTOR_T
>  typedef u64 sector_t;
> +#define SECTOR_FMT "%llu"

We did try this a few years ago, but I cannot for the life of me remember
our reasons for abandoning it :(

We ended up deciding that the best way of handling this is to open-code the
%lld and to typecast the argument to long long.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2006-06-09  2:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-06-09  1:22 [RFC 5/13] extents and 48bit ext3: sector_t type format string Mingming Cao
2006-06-09  2:45 ` Andrew Morton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).