On Fri, 17 October 2008 16:42:50 +0100, Phillip Lougher wrote: > > Codewise all of the packed bit-fields and the swap macros have been removed in > favour of aligned structures and in-line swapping using leXX_to_cpu(). The > code has also been extensively restructured, reformatted to kernel coding > standards and commented. Excellent! The data structures look good and I don't see a reason for another format change. Which means the main reason against merging the code has gone. Your style differs from other kernel code and in a number of cases it would be nice to be more consistent with existing conventions. It would certainly help others when reading the code. And of course, one way to do so it to just merge and wait for some janitors to notice squashfs and send patches. :) I have to admit I am scared of this function: +int squashfs_read_metadata(struct super_block *s, void *buffer, + long long block, unsigned int offset, + int length, long long *next_block, + unsigned int *next_offset) It takes seven parameters, five of which look deceptively similar to me. Almost every time I see a call to this function, my mind goes blank. There must be some way to make this function a bit more agreeable. One option is to fuse the "block" and "offset" parameters into a struct and just pass two sets of this struct. Another would be to combine the two sets of addresses into a single one. A quick look at some of the callers shows seems to favor that approach. squashfs_read_metadata(..., block, offset, ..., &block, &offset) Could become squashfs_read_metadata(..., &block, &offset, ...) But again, such a change is no showstopper for mainline inclusion. > Anyway that's my case for inclusion. If any readers want Squashfs > mainlined it's probably now a good time to offer support! Please no. A large amount of popular support would only bring you into the reiser4 league. Bad arguments don't improve when repeated. Support in the form of patches would be a different matter, though. Jörn -- Mac is for working, Linux is for Networking, Windows is for Solitaire! -- stolen from dc -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
