On Thu, 2011-06-02 at 15:04 -0700, H Hartley Sweeten wrote: > There are many f_op->open implementations that simply do: > > file->private_data = inode->i_private; > > This introduces a new global symbol, default_open, that can be > used instead of the private implementations. > > This also removes a couple private implementations that share the > same symbol name. > > Signed-off-by: H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx> > Cc: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx> > Cc: Matthew Wilcox <matthew@xxxxxx> > Cc: Artem Bityutskiy <dedekind1@xxxxxxxxx> > Cc: David Woodhouse <dwmw2@xxxxxxxxxxxxx> > Cc: Robert Richter <robert.richter@xxxxxxx> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxx> > Cc: James Morris <jmorris@xxxxxxxxx> > Cc: Nick Piggin <npiggin@xxxxxxxxx> > Cc: Andi Kleen <ak@xxxxxxxxxxxxxxx> > Cc: Dave Chinner <dchinner@xxxxxxxxxx> > > --- > > If this patch looks reasonable, I have some additional patches that > convert all other similar f_op->open implemenations to use default_open > tree wide. The full diffstat looks like this: Looks good to me, thanks. -- Best Regards, Artem Bityutskiy (ÐÑÑÑÐ ÐÐÑÑÑÐÐÐ) -- 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