Re: [PATCH 3/6] xfs: remove obsolete osyncisosync mount option

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Tue, Jul 20, 2010 at 05:38:32PM +1000, Dave Chinner wrote:
> >     "XFS: ihashsize no longer used, option is deprecated.");
> >             } else if (!strcmp(this_char, "osyncisdsync")) {
> > -                   /* no-op, this is now the default */
> >                     cmn_err(CE_WARN,
> > -   "XFS: osyncisdsync is now the default, option is deprecated.");
> > +   "XFS: the osyncisdsync option is deprecated as has no effect.");
> > +           } else if (!strcmp(this_char, "osyncisosync")) {
> > +                   cmn_err(CE_WARN,
> > +   "XFS: the osyncisosync option is deprecated as has no effect.");
> >             } else if (!strcmp(this_char, "irixsgid")) {
> >                     cmn_err(CE_WARN,
> >     "XFS: irixsgid is now a sysctl(2) variable, option is deprecated.");
> 
> Look better as:
> 
>       "XFS: osyncisdsync has no effect, option is deprecated."
>       "XFS: osyncisosync has no effect, option is deprecated."
> 
> so they are consistent with the other deprecated option output.
> I've made this change to the patch i have in my tree....

Sure, fine with me.