On Mon, May 07, 2012 at 04:21:30PM -0700, Srinivas Eeda wrote:
> Currently reservation code assumes a bitmap given to it is all one contigous
> chunk. This patch enhances it to handle a discontigous chunks. It adds new
> fields m_bitmap_ext_cnt and m_bitmap_ext_arr. m_bitmap_ext_arr tracks the
> sizes
> of each contigous free bits and m_bitmap_ext_cnt trackes number of
> m_bitmap_ext_arr.
>
> Signed-off-by: Srinivas Eeda <srinivas.e...@oracle.com>
Hi Srini,
A patch like this should come before the feature patch. Once
this code can treat the old single-range bitmap as a one-element
multiple-range bitmap, you can add the multiple-range change easily.
> +void ocfs2_resmap_set_ext(struct ocfs2_reservation_map *resmap, int arr, u32
> sz)
> +{
> + if (ocfs2_resmap_disabled(resmap))
> + return;
> +
> + resmap->m_bitmap_ext_arr[arr] = sz;
> +}
I don't see this function called anywhere. And please don't use
needless abbreviations. If you want to say ocfs2_resmap_set_extent(),
write it out. I don't quite get the arguments, and since it isn't
called, I can't figure out how they are used.
Joel
--
"To announce that there must be no criticism of them president, or
that we are to stand by the president, right or wrong, is not only
unpatriotic and servile, but is morally treasonable to the American
public."
- Theodore Roosevelt
http://www.jlbec.org/
jl...@evilplan.org
_______________________________________________
Ocfs2-devel mailing list
Ocfs2-devel@oss.oracle.com
http://oss.oracle.com/mailman/listinfo/ocfs2-devel
