Arnd Bergmann <arnd@xxxxxxxx> wrote: > It also makes things like strace more complicated. That's the most compelling argument. > No, I think that would be worse than the current version. But if you remove > the structure version in favor of the flags, you only need six arguments > anyway. I want to keep the structure version, just in case we need to expand fields in the stat struct in future. Otherwise we may need to create yet another stat syscall. > You can also go further and fold the structure length into flags, because > the length is just a function of the data you are passing. The potential problem with passing the flags as a syscall argument is that we're then limited to a single 32-bit integer. It might be enough, but if I do as at least one person has suggested and assign each field in the struct its own bit, that uses up half right there, plus I'd like to add at least one operational flag (to force synchronisation with the server). > Having a system call with flags, size and version is like wearing a belt, > braces and suspenders. An unsigned long flags argument should be enough to > hold up your pants[1]. I would like the size argument for two reasons: firstly, to prevent buffer overruns and, secondly, because I can see some scope for variable-size fields (such as for volume IDs or security labels), though the latter might be better handled through getxattr() (which would mean extra overhead). David -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
