[XFS] Undo bit ops cleanup mod due to regression on 32-bit powermac

platform. SGI-PV: 971186 SGI-Modid: xfs-linux-melb:xfs-kern:30559a Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
author: Lachlan McIlroy <lachlan@sgi.com> 2008-02-26 01:00:22 -0500
committer: Lachlan McIlroy <lachlan@redback.melbourne.sgi.com> 2008-02-26 01:05:44 -0500
commit: ef8ece55d9b6825c28a5c1a4bd89b94040cb7b32 (patch)
tree: c086be0bf7ef1139db75d4fd084f2390545629e8 /fs
parent: db69c915e67705daac25cad06d816c09be634de0 (diff)
3 files changed, 120 insertions, 29 deletions
diff --git a/fs/xfs/xfs_bit.c b/fs/xfs/xfs_bit.c
index 48228848f5ae..fab0b6d5a41b 100644
--- a/fs/xfs/xfs_bit.c
+++ b/fs/xfs/xfs_bit.c
@@ -25,6 +25,109 @@
 * XFS bit manipulation routines, used in non-realtime code.
 */
+#ifndef HAVE_ARCH_HIGHBIT
+/*
+ * Index of high bit number in byte, -1 for none set, 0..7 otherwise.
+ */
+static const char xfs_highbit[256] = {
+       -1, 0, 1, 1, 2, 2, 2, 2,                 /* 00 .. 07 */
+        3, 3, 3, 3, 3, 3, 3, 3,                 /* 08 .. 0f */
+        4, 4, 4, 4, 4, 4, 4, 4,                 /* 10 .. 17 */
+        4, 4, 4, 4, 4, 4, 4, 4,                 /* 18 .. 1f */
+        5, 5, 5, 5, 5, 5, 5, 5,                 /* 20 .. 27 */
+        5, 5, 5, 5, 5, 5, 5, 5,                 /* 28 .. 2f */
+        5, 5, 5, 5, 5, 5, 5, 5,                 /* 30 .. 37 */
+        5, 5, 5, 5, 5, 5, 5, 5,                 /* 38 .. 3f */
+        6, 6, 6, 6, 6, 6, 6, 6,                 /* 40 .. 47 */
+        6, 6, 6, 6, 6, 6, 6, 6,                 /* 48 .. 4f */
+        6, 6, 6, 6, 6, 6, 6, 6,                 /* 50 .. 57 */
+        6, 6, 6, 6, 6, 6, 6, 6,                 /* 58 .. 5f */
+        6, 6, 6, 6, 6, 6, 6, 6,                 /* 60 .. 67 */
+        6, 6, 6, 6, 6, 6, 6, 6,                 /* 68 .. 6f */
+        6, 6, 6, 6, 6, 6, 6, 6,                 /* 70 .. 77 */
+        6, 6, 6, 6, 6, 6, 6, 6,                 /* 78 .. 7f */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* 80 .. 87 */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* 88 .. 8f */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* 90 .. 97 */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* 98 .. 9f */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* a0 .. a7 */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* a8 .. af */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* b0 .. b7 */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* b8 .. bf */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* c0 .. c7 */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* c8 .. cf */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* d0 .. d7 */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* d8 .. df */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* e0 .. e7 */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* e8 .. ef */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* f0 .. f7 */
+        7, 7, 7, 7, 7, 7, 7, 7,                 /* f8 .. ff */
+};
+#endif
+/*
+ * xfs_highbit32: get high bit set out of 32-bit argument, -1 if none set.
+ */
+inline int
+xfs_highbit32(
+        __uint32_t      v)
+{
+#ifdef HAVE_ARCH_HIGHBIT
+        return highbit32(v);
+#else
+        int             i;
+        if (v & 0xffff0000)
+                if (v & 0xff000000)
+                        i = 24;
+                else
+                        i = 16;
+        else if (v & 0x0000ffff)
+                if (v & 0x0000ff00)
+                        i = 8;
+                else
+                        i = 0;
+        else
+                return -1;
+        return i + xfs_highbit[(v >> i) & 0xff];
+#endif
+}
+/*
+ * xfs_lowbit64: get low bit set out of 64-bit argument, -1 if none set.
+ */
+int
+xfs_lowbit64(
+        __uint64_t      v)
+{
+        __uint32_t      w = (__uint32_t)v;
+        int             n = 0;
+        if (w) {        /* lower bits */
+                n = ffs(w);
+        } else {        /* upper bits */
+                w = (__uint32_t)(v >> 32);
+                if (w && (n = ffs(w)))
+                        n += 32;
+        }
+        return n - 1;
+}
+/*
+ * xfs_highbit64: get high bit set out of 64-bit argument, -1 if none set.
+ */
+int
+xfs_highbit64(
+        __uint64_t      v)
+{
+        __uint32_t      h = (__uint32_t)(v >> 32);
+        if (h)
+                return xfs_highbit32(h) + 32;
+        return xfs_highbit32((__uint32_t)v);
+}
 /*
 * Return whether bitmap is empty.
 * Size is number of words in the bitmap, which is padded to word boundary
diff --git a/fs/xfs/xfs_bit.h b/fs/xfs/xfs_bit.h
index 0f9fc9a3c415..082641a9782c 100644
--- a/fs/xfs/xfs_bit.h
+++ b/fs/xfs/xfs_bit.h
@@ -47,30 +47,13 @@ static inline __uint64_t xfs_mask64lo(int n)
 }
 /* Get high bit set out of 32-bit argument, -1 if none set */
-static inline int xfs_highbit32(__uint32_t v)
+extern int xfs_highbit32(__uint32_t v);
-{
-        return fls(v) - 1;
-}
-/* Get high bit set out of 64-bit argument, -1 if none set */
-static inline int xfs_highbit64(__uint64_t v)
-{
-        return fls64(v) - 1;
-}
-/* Get low bit set out of 32-bit argument, -1 if none set */
-static inline int xfs_lowbit32(__uint32_t v)
-{
-        unsigned long   t = v;
-        return (v) ? find_first_bit(&t, 32) : -1;
-}
 /* Get low bit set out of 64-bit argument, -1 if none set */
-static inline int xfs_lowbit64(__uint64_t v)
+extern int xfs_lowbit64(__uint64_t v);
-{
-        unsigned long   t = v;
+/* Get high bit set out of 64-bit argument, -1 if none set */
-        return (v) ? find_first_bit(&t, 64) : -1;
+extern int xfs_highbit64(__uint64_t);
-}
 /* Return whether bitmap is empty (1 == empty) */
 extern int xfs_bitmap_empty(uint *map, uint size);
diff --git a/fs/xfs/xfs_rtalloc.c b/fs/xfs/xfs_rtalloc.c
index ca83ddf72af4..47082c01872d 100644
--- a/fs/xfs/xfs_rtalloc.c
+++ b/fs/xfs/xfs_rtalloc.c
@@ -73,6 +73,18 @@ STATIC int xfs_rtmodify_summary(xfs_mount_t *, xfs_trans_t *, int,
 */
 /*
+ * xfs_lowbit32: get low bit set out of 32-bit argument, -1 if none set.
+ */
+STATIC int
+xfs_lowbit32(
+        __uint32_t      v)
+{
+        if (v)
+                return ffs(v) - 1;
+        return -1;
+}
+/*
 * Allocate space to the bitmap or summary file, and zero it, for growfs.
 */
 STATIC int                              /* error */
@@ -432,7 +444,6 @@ xfs_rtallocate_extent_near(
        }
        bbno = XFS_BITTOBLOCK(mp, bno);
        i = 0;
-        ASSERT(minlen != 0);
        log2len = xfs_highbit32(minlen);
        /*
         * Loop over all bitmap blocks (bbno + i is current block).
@@ -601,8 +612,6 @@ xfs_rtallocate_extent_size(
        xfs_suminfo_t   sum;            /* summary information for extents */
        ASSERT(minlen % prod == 0 && maxlen % prod == 0);
-        ASSERT(maxlen != 0);
        /*
         * Loop over all the levels starting with maxlen.
         * At each level, look at all the bitmap blocks, to see if there
@@ -660,9 +669,6 @@ xfs_rtallocate_extent_size(
                *rtblock = NULLRTBLOCK;
                return 0;
        }
-        ASSERT(minlen != 0);
-        ASSERT(maxlen != 0);
        /*
         * Loop over sizes, from maxlen down to minlen.
         * This time, when we do the allocations, allow smaller ones
@@ -1948,7 +1954,6 @@ xfs_growfs_rt(
                                  nsbp->sb_blocksize * nsbp->sb_rextsize);
                nsbp->sb_rextents = nsbp->sb_rblocks;
                do_div(nsbp->sb_rextents, nsbp->sb_rextsize);
-                ASSERT(nsbp->sb_rextents != 0);
                nsbp->sb_rextslog = xfs_highbit32(nsbp->sb_rextents);
                nrsumlevels = nmp->m_rsumlevels = nsbp->sb_rextslog + 1;
                nrsumsize =
author	Lachlan McIlroy <lachlan@sgi.com>	2008-02-26 01:00:22 -0500
committer	Lachlan McIlroy <lachlan@redback.melbourne.sgi.com>	2008-02-26 01:05:44 -0500
commit	ef8ece55d9b6825c28a5c1a4bd89b94040cb7b32 (patch)
tree	c086be0bf7ef1139db75d4fd084f2390545629e8 /fs
parent	db69c915e67705daac25cad06d816c09be634de0 (diff)

diff --git a/fs/xfs/xfs_bit.c b/fs/xfs/xfs_bit.c index 48228848f5ae..fab0b6d5a41b 100644 --- a/fs/xfs/xfs_bit.c +++ b/fs/xfs/xfs_bit.c
@@ -25,6 +25,109 @@
25	* XFS bit manipulation routines, used in non-realtime code.	25	* XFS bit manipulation routines, used in non-realtime code.
26	*/	26	*/
27		27
		28	#ifndef HAVE_ARCH_HIGHBIT
		29	/*
		30	* Index of high bit number in byte, -1 for none set, 0..7 otherwise.
		31	*/
		32	static const char xfs_highbit[256] = {
		33	-1, 0, 1, 1, 2, 2, 2, 2, /* 00 .. 07 */
		34	3, 3, 3, 3, 3, 3, 3, 3, /* 08 .. 0f */
		35	4, 4, 4, 4, 4, 4, 4, 4, /* 10 .. 17 */
		36	4, 4, 4, 4, 4, 4, 4, 4, /* 18 .. 1f */
		37	5, 5, 5, 5, 5, 5, 5, 5, /* 20 .. 27 */
		38	5, 5, 5, 5, 5, 5, 5, 5, /* 28 .. 2f */
		39	5, 5, 5, 5, 5, 5, 5, 5, /* 30 .. 37 */
		40	5, 5, 5, 5, 5, 5, 5, 5, /* 38 .. 3f */
		41	6, 6, 6, 6, 6, 6, 6, 6, /* 40 .. 47 */
		42	6, 6, 6, 6, 6, 6, 6, 6, /* 48 .. 4f */
		43	6, 6, 6, 6, 6, 6, 6, 6, /* 50 .. 57 */
		44	6, 6, 6, 6, 6, 6, 6, 6, /* 58 .. 5f */
		45	6, 6, 6, 6, 6, 6, 6, 6, /* 60 .. 67 */
		46	6, 6, 6, 6, 6, 6, 6, 6, /* 68 .. 6f */
		47	6, 6, 6, 6, 6, 6, 6, 6, /* 70 .. 77 */
		48	6, 6, 6, 6, 6, 6, 6, 6, /* 78 .. 7f */
		49	7, 7, 7, 7, 7, 7, 7, 7, /* 80 .. 87 */
		50	7, 7, 7, 7, 7, 7, 7, 7, /* 88 .. 8f */
		51	7, 7, 7, 7, 7, 7, 7, 7, /* 90 .. 97 */
		52	7, 7, 7, 7, 7, 7, 7, 7, /* 98 .. 9f */
		53	7, 7, 7, 7, 7, 7, 7, 7, /* a0 .. a7 */
		54	7, 7, 7, 7, 7, 7, 7, 7, /* a8 .. af */
		55	7, 7, 7, 7, 7, 7, 7, 7, /* b0 .. b7 */
		56	7, 7, 7, 7, 7, 7, 7, 7, /* b8 .. bf */
		57	7, 7, 7, 7, 7, 7, 7, 7, /* c0 .. c7 */
		58	7, 7, 7, 7, 7, 7, 7, 7, /* c8 .. cf */
		59	7, 7, 7, 7, 7, 7, 7, 7, /* d0 .. d7 */
		60	7, 7, 7, 7, 7, 7, 7, 7, /* d8 .. df */
		61	7, 7, 7, 7, 7, 7, 7, 7, /* e0 .. e7 */
		62	7, 7, 7, 7, 7, 7, 7, 7, /* e8 .. ef */
		63	7, 7, 7, 7, 7, 7, 7, 7, /* f0 .. f7 */
		64	7, 7, 7, 7, 7, 7, 7, 7, /* f8 .. ff */
		65	};
		66	#endif
		67
		68	/*
		69	* xfs_highbit32: get high bit set out of 32-bit argument, -1 if none set.
		70	*/
		71	inline int
		72	xfs_highbit32(
		73	__uint32_t v)
		74	{
		75	#ifdef HAVE_ARCH_HIGHBIT
		76	return highbit32(v);
		77	#else
		78	int i;
		79
		80	if (v & 0xffff0000)
		81	if (v & 0xff000000)
		82	i = 24;
		83	else
		84	i = 16;
		85	else if (v & 0x0000ffff)
		86	if (v & 0x0000ff00)
		87	i = 8;
		88	else
		89	i = 0;
		90	else
		91	return -1;
		92	return i + xfs_highbit[(v >> i) & 0xff];
		93	#endif
		94	}
		95
		96	/*
		97	* xfs_lowbit64: get low bit set out of 64-bit argument, -1 if none set.
		98	*/
		99	int
		100	xfs_lowbit64(
		101	__uint64_t v)
		102	{
		103	__uint32_t w = (__uint32_t)v;
		104	int n = 0;
		105
		106	if (w) { /* lower bits */
		107	n = ffs(w);
		108	} else { /* upper bits */
		109	w = (__uint32_t)(v >> 32);
		110	if (w && (n = ffs(w)))
		111	n += 32;
		112	}
		113	return n - 1;
		114	}
		115
		116	/*
		117	* xfs_highbit64: get high bit set out of 64-bit argument, -1 if none set.
		118	*/
		119	int
		120	xfs_highbit64(
		121	__uint64_t v)
		122	{
		123	__uint32_t h = (__uint32_t)(v >> 32);
		124
		125	if (h)
		126	return xfs_highbit32(h) + 32;
		127	return xfs_highbit32((__uint32_t)v);
		128	}
		129
		130
28	/*	131	/*
29	* Return whether bitmap is empty.	132	* Return whether bitmap is empty.
30	* Size is number of words in the bitmap, which is padded to word boundary	133	* Size is number of words in the bitmap, which is padded to word boundary


diff --git a/fs/xfs/xfs_bit.h b/fs/xfs/xfs_bit.h index 0f9fc9a3c415..082641a9782c 100644 --- a/fs/xfs/xfs_bit.h +++ b/fs/xfs/xfs_bit.h
@@ -47,30 +47,13 @@ static inline __uint64_t xfs_mask64lo(int n)
47	}	47	}
48		48
49	/* Get high bit set out of 32-bit argument, -1 if none set */	49	/* Get high bit set out of 32-bit argument, -1 if none set */
50	static inline int xfs_highbit32(__uint32_t v)	50	extern int xfs_highbit32(__uint32_t v);
51	{
52	return fls(v) - 1;
53	}
54
55	/* Get high bit set out of 64-bit argument, -1 if none set */
56	static inline int xfs_highbit64(__uint64_t v)
57	{
58	return fls64(v) - 1;
59	}
60
61	/* Get low bit set out of 32-bit argument, -1 if none set */
62	static inline int xfs_lowbit32(__uint32_t v)
63	{
64	unsigned long t = v;
65	return (v) ? find_first_bit(&t, 32) : -1;
66	}
67		51
68	/* Get low bit set out of 64-bit argument, -1 if none set */	52	/* Get low bit set out of 64-bit argument, -1 if none set */
69	static inline int xfs_lowbit64(__uint64_t v)	53	extern int xfs_lowbit64(__uint64_t v);
70	{	54
71	unsigned long t = v;	55	/* Get high bit set out of 64-bit argument, -1 if none set */
72	return (v) ? find_first_bit(&t, 64) : -1;	56	extern int xfs_highbit64(__uint64_t);
73	}
74		57
75	/* Return whether bitmap is empty (1 == empty) */	58	/* Return whether bitmap is empty (1 == empty) */
76	extern int xfs_bitmap_empty(uint *map, uint size);	59	extern int xfs_bitmap_empty(uint *map, uint size);


diff --git a/fs/xfs/xfs_rtalloc.c b/fs/xfs/xfs_rtalloc.c index ca83ddf72af4..47082c01872d 100644 --- a/fs/xfs/xfs_rtalloc.c +++ b/fs/xfs/xfs_rtalloc.c
@@ -73,6 +73,18 @@ STATIC int xfs_rtmodify_summary(xfs_mount_t , xfs_trans_t , int,
73	*/	73	*/
74		74
75	/*	75	/*
		76	* xfs_lowbit32: get low bit set out of 32-bit argument, -1 if none set.
		77	*/
		78	STATIC int
		79	xfs_lowbit32(
		80	__uint32_t v)
		81	{
		82	if (v)
		83	return ffs(v) - 1;
		84	return -1;
		85	}
		86
		87	/*
76	* Allocate space to the bitmap or summary file, and zero it, for growfs.	88	* Allocate space to the bitmap or summary file, and zero it, for growfs.
77	*/	89	*/
78	STATIC int /* error */	90	STATIC int /* error */
@@ -432,7 +444,6 @@ xfs_rtallocate_extent_near(
432	}	444	}
433	bbno = XFS_BITTOBLOCK(mp, bno);	445	bbno = XFS_BITTOBLOCK(mp, bno);
434	i = 0;	446	i = 0;
435	ASSERT(minlen != 0);
436	log2len = xfs_highbit32(minlen);	447	log2len = xfs_highbit32(minlen);
437	/*	448	/*
438	* Loop over all bitmap blocks (bbno + i is current block).	449	* Loop over all bitmap blocks (bbno + i is current block).
@@ -601,8 +612,6 @@ xfs_rtallocate_extent_size(
601	xfs_suminfo_t sum; /* summary information for extents */	612	xfs_suminfo_t sum; /* summary information for extents */
602		613
603	ASSERT(minlen % prod == 0 && maxlen % prod == 0);	614	ASSERT(minlen % prod == 0 && maxlen % prod == 0);
604	ASSERT(maxlen != 0);
605
606	/*	615	/*
607	* Loop over all the levels starting with maxlen.	616	* Loop over all the levels starting with maxlen.
608	* At each level, look at all the bitmap blocks, to see if there	617	* At each level, look at all the bitmap blocks, to see if there
@@ -660,9 +669,6 @@ xfs_rtallocate_extent_size(
660	*rtblock = NULLRTBLOCK;	669	*rtblock = NULLRTBLOCK;
661	return 0;	670	return 0;
662	}	671	}
663	ASSERT(minlen != 0);
664	ASSERT(maxlen != 0);
665
666	/*	672	/*
667	* Loop over sizes, from maxlen down to minlen.	673	* Loop over sizes, from maxlen down to minlen.
668	* This time, when we do the allocations, allow smaller ones	674	* This time, when we do the allocations, allow smaller ones
@@ -1948,7 +1954,6 @@ xfs_growfs_rt(
1948	nsbp->sb_blocksize * nsbp->sb_rextsize);	1954	nsbp->sb_blocksize * nsbp->sb_rextsize);
1949	nsbp->sb_rextents = nsbp->sb_rblocks;	1955	nsbp->sb_rextents = nsbp->sb_rblocks;
1950	do_div(nsbp->sb_rextents, nsbp->sb_rextsize);	1956	do_div(nsbp->sb_rextents, nsbp->sb_rextsize);
1951	ASSERT(nsbp->sb_rextents != 0);
1952	nsbp->sb_rextslog = xfs_highbit32(nsbp->sb_rextents);	1957	nsbp->sb_rextslog = xfs_highbit32(nsbp->sb_rextents);
1953	nrsumlevels = nmp->m_rsumlevels = nsbp->sb_rextslog + 1;	1958	nrsumlevels = nmp->m_rsumlevels = nsbp->sb_rextslog + 1;
1954	nrsumsize =	1959	nrsumsize =