2011/08/18 1:31, Andreas Dilger wrote:
It is confusing in conditionals like this when integer variables are treated
as boolean values. It would be more clear to compare the orig_score and
donor_score to zero.
if (orig_score == 0 || (donor_score > 0 && !(mode_flag & FORCE)) ||
(orig_score <= donor_score && (mode_flag & FORCE))) {
To be honest, I still can't understand the above logic. I would think it is
enough to check:
if (orig_score < donor_score && !(mode_flag & FORCE))) {
so that the file is not defragged if the score would get worse, but "force"
means it is always moved.
e4defrag has two conditions to call EXT4_IOC_MOVE_EXT ioctl:
(1) the original file is fragmented file (orig_score > 0)
(2) the donor file is not fragmented file (donor_score == 0)
It could be that the donor file is a little fragmented file in case of few disk
space available, so sometimes we cannot defrag a file due to the condition (2).
However, it makes no sense that the fragmentation gets worse due to
e4defrag. Hence, I added a new condition (orig_score > donor_score) instead of
the condition (2) to check whether the fragmentation gets better.
The condition (1) is necessary for e4defarg -F. Because if e2p_get_fragscore()
returns zero in case of maximum threshold (2^32-1), it means the number of
the extents doesn't decrease any more even if the file gets the best extent
mapping.
printf("\033[79;0H\033[K[%u/%u]%s:\t%3d%%",
It would be nice if the color terminal output was optional (maybe only on
by default for tty output). This is not typical for other e2fsprogs utilities,
and makes a mess of logs being kept of the output.
I see. I'll fix the output based on mke2fs.
Regards,
Kazuya Mio
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
