On 2011-12-18, at 15:24, Tao Ma [off-list ref] wrote:

From: Tao Ma <redacted>

Currently, in ext4_iget we do a simple check to see whether
there does exist some information starting from the end
of i_extra_size. With inline data added, this procedure
is more complicated. So move it to a new function named
ext4_iget_extra_inode.

+        } else
+            ext4_iget_extra_inode(inode, raw_inode, ei);
   } else

(style) if one half of an if-else uses braces then the other half should as well. 

Cheers, Andreas