On Wed, Jan 28, 2009 at 10:45:47PM -0800, Sukadev Bhattiprolu wrote: > + fsi = DEVPTS_SB(s); > + opts = &fsi->mount_opts; > + parse_mount_options(data, PARSE_REMOUNT, opts); You could probably get rid of at least the fsi variable without making the code less readable. Independent of these cosmetic bits the patch looks good to me. _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers