On Tue, Sep 29, 2009 at 02:18:39PM +0900, Akira Fujita wrote: > I thought the argument check (orig and donor inodes are different) > should be done in mext_check_arguments() Sometimes separating things into multiple functions can actually make things harder to understand, since it means you have to jump all over the file to follow what a particular function is doing. Sometimes having a function which is nested deeply in a loop can be made easier to understand by separating it out into a separate function which is called once. Part of the reason why mext_check_arguments() is so huge is because of all of the ext4_debug() statements. It's not clear to me they really all need to be there. I'd also use EBUSY if the file is a swap file, so it's easier for the userspace code to understand what is going on in that case. For the other many cases where EINVAL is returned, hopefully that's obvious enough for userspace to figure out without having to resort to looking to the system debug logs. - Ted -- 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