On Thu, Oct 24, 2019 at 03:51:01PM +0800, Ian Kent wrote: > -#ifndef CONFIG_XFS_QUOTA > - if (XFS_IS_QUOTA_RUNNING(mp)) { > +#if !IS_ENABLED(CONFIG_XFS_QUOTA) > + if (mp->m_qflags != 0) { The whole point of IS_ENABLED is that you can directly use it in C code instead of only through the preprocessor. So this should be: if (!IS_ENABLED(CONFIG_XFS_QUOTA) && mp->m_qflags != 0) {