]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
ocfs2: make ocfs2_adjust_resv_from_alloc simple.
authorTao Ma <tao.ma@oracle.com>
Thu, 8 Apr 2010 08:33:02 +0000 (16:33 +0800)
committerJoel Becker <joel.becker@oracle.com>
Thu, 6 May 2010 01:18:09 +0000 (18:18 -0700)
commitb065556a7d1a9205403db77a318a5c5aa530e701
treefd8ef1e5e67c624c9fb04689e4a4765e2e59acf8
parent4b37fcb7d41ce3b9264b9562d6ffd62db9294bd1
ocfs2: make ocfs2_adjust_resv_from_alloc simple.

When we allocate some bits from the reservation, we always
allocate from the r_start(see ocfs2_resmap_resv_bits).
So there should be no reason to check between r_start
and start. And I don't think we will change this behaviour
later by allocating from some bits after r_start.  Why not make
ocfs2_adjust_resv_from_alloc simple for now?

The only chance we have to adjust the reservation is when we haven't
reached the end. With this patch, the function is more readable.

Note:
btw, this patch also fixes an original bug in the function
which I haven't found before.
if (end < ocfs2_resv_end(resv))
rhs = end - ocfs2_resv_end(resv);
This code is of course buggy. ;)

Signed-off-by: Tao Ma <tao.ma@oracle.com>
Acked-by: Mark Fasheh <mfasheh@suse.com>
Signed-off-by: Joel Becker <joel.becker@oracle.com>
fs/ocfs2/reservations.c
fs/ocfs2/reservations.h