On Mon, Jul 03, 2006 at 10:06:42AM +0100, Christoph Hellwig wrote:On Sun, Jul 02, 2006 at 08:53:40PM -0400, Theodore Ts'o wrote:The i_blocks and i_size fields are only used for regular files. So weCan we please make this a named instead of unnamed union so everyone still
move them into the union, along with i_rdev and i_devices, which are
only used by block or character devices.
using the fields will trip up? To reduce the impact a few more imajor/iminor
conversions might be needed were direct references to i_rdev crept back in.
I did that originally but when I sent out my first version of patches
for review, other developers asked me to use an unnamed union ---
since otherwise the patch would be much, much larger (lots of changes
would need to be made) and that makes it much harder to merge into
either Andrew's or Linus's tree.
What do other people think? I can go either way on this one.