Commit 792314f8 authored by Russell King's avatar Russell King Committed by Al Viro

fs/adfs: map: use find_next_bit_le() rather than open coding it

Use find_next_bit_le() to find the end of a fragment in the map rather
than open-coding this functionality.
Signed-off-by: default avatarRussell King <rmk+kernel@armlinux.org.uk>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 197ba3c5
...@@ -72,50 +72,32 @@ static int lookup_zone(const struct adfs_discmap *dm, const unsigned int idlen, ...@@ -72,50 +72,32 @@ static int lookup_zone(const struct adfs_discmap *dm, const unsigned int idlen,
const u32 idmask = (1 << idlen) - 1; const u32 idmask = (1 << idlen) - 1;
unsigned char *map = dm->dm_bh->b_data; unsigned char *map = dm->dm_bh->b_data;
unsigned int start = dm->dm_startbit; unsigned int start = dm->dm_startbit;
unsigned int mapptr; unsigned int fragend;
u32 frag; u32 frag;
do { do {
frag = GET_FRAG_ID(map, start, idmask); frag = GET_FRAG_ID(map, start, idmask);
mapptr = start + idlen;
fragend = find_next_bit_le(map, endbit, start + idlen);
/* if (fragend >= endbit)
* find end of fragment goto error;
*/
{ if (frag == frag_id) {
__le32 *_map = (__le32 *)map; unsigned int length = fragend + 1 - start;
u32 v = le32_to_cpu(_map[mapptr >> 5]) >> (mapptr & 31);
while (v == 0) { if (*offset < length)
mapptr = (mapptr & ~31) + 32; return start + *offset;
if (mapptr >= endbit) *offset -= length;
goto error;
v = le32_to_cpu(_map[mapptr >> 5]);
}
mapptr += 1 + ffz(~v);
} }
if (frag == frag_id) start = fragend + 1;
goto found; } while (start < endbit);
again:
start = mapptr;
} while (mapptr < endbit);
return -1; return -1;
error: error:
printk(KERN_ERR "adfs: oversized fragment 0x%x at 0x%x-0x%x\n", printk(KERN_ERR "adfs: oversized fragment 0x%x at 0x%x-0x%x\n",
frag, start, mapptr); frag, start, fragend);
return -1; return -1;
found:
{
int length = mapptr - start;
if (*offset >= length) {
*offset -= length;
goto again;
}
}
return start + *offset;
} }
/* /*
...@@ -132,7 +114,7 @@ scan_free_map(struct adfs_sb_info *asb, struct adfs_discmap *dm) ...@@ -132,7 +114,7 @@ scan_free_map(struct adfs_sb_info *asb, struct adfs_discmap *dm)
const unsigned int frag_idlen = idlen <= 15 ? idlen : 15; const unsigned int frag_idlen = idlen <= 15 ? idlen : 15;
const u32 idmask = (1 << frag_idlen) - 1; const u32 idmask = (1 << frag_idlen) - 1;
unsigned char *map = dm->dm_bh->b_data; unsigned char *map = dm->dm_bh->b_data;
unsigned int start = 8, mapptr; unsigned int start = 8, fragend;
u32 frag; u32 frag;
unsigned long total = 0; unsigned long total = 0;
...@@ -151,29 +133,13 @@ scan_free_map(struct adfs_sb_info *asb, struct adfs_discmap *dm) ...@@ -151,29 +133,13 @@ scan_free_map(struct adfs_sb_info *asb, struct adfs_discmap *dm)
do { do {
start += frag; start += frag;
/*
* get fragment id
*/
frag = GET_FRAG_ID(map, start, idmask); frag = GET_FRAG_ID(map, start, idmask);
mapptr = start + idlen;
/*
* find end of fragment
*/
{
__le32 *_map = (__le32 *)map;
u32 v = le32_to_cpu(_map[mapptr >> 5]) >> (mapptr & 31);
while (v == 0) {
mapptr = (mapptr & ~31) + 32;
if (mapptr >= endbit)
goto error;
v = le32_to_cpu(_map[mapptr >> 5]);
}
mapptr += 1 + ffz(~v);
}
total += mapptr - start; fragend = find_next_bit_le(map, endbit, start + idlen);
if (fragend >= endbit)
goto error;
total += fragend + 1 - start;
} while (frag >= idlen + 1); } while (frag >= idlen + 1);
if (frag != 0) if (frag != 0)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment