From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cuda.sgi.com (cuda2.sgi.com [192.48.176.25]) by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id q7H2NTvA232910 for ; Thu, 16 Aug 2012 21:23:30 -0500 Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by cuda.sgi.com with ESMTP id nL754UG6OCnV2FNH for ; Thu, 16 Aug 2012 19:23:29 -0700 (PDT) Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q7H2NSv0021054 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Thu, 16 Aug 2012 22:23:28 -0400 Received: from orion.usersys.redhat.com (vpn1-6-11.gru2.redhat.com [10.97.6.11]) by int-mx12.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id q7H2NPQI019079 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO) for ; Thu, 16 Aug 2012 22:23:27 -0400 Date: Thu, 16 Aug 2012 23:23:24 -0300 From: Carlos Maiolino Subject: Re: [PATCH V2] Make inode64 a remountable option Message-ID: <20120817022324.GC16013@orion.usersys.redhat.com> References: <1345169925-18927-1-git-send-email-cmaiolino@redhat.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1345169925-18927-1-git-send-email-cmaiolino@redhat.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: xfs-bounces@oss.sgi.com Errors-To: xfs-bounces@oss.sgi.com To: xfs@oss.sgi.com Sorry, added a wrong element in the loop, sending a V3 now. On Thu, Aug 16, 2012 at 11:18:45PM -0300, Carlos Maiolino wrote: > Actually, there is no reason about why a user must umount and mount a XFS > filesystem to enable 'inode64' option. So, this patch makes this a remountable > option. > > Signed-off-by: Carlos Maiolino > --- > fs/xfs/xfs_super.c | 18 +++++++++++++++++- > 1 file changed, 17 insertions(+), 1 deletion(-) > > diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c > index bdaf4cb..d5ef8c7 100644 > --- a/fs/xfs/xfs_super.c > +++ b/fs/xfs/xfs_super.c > @@ -120,12 +120,13 @@ mempool_t *xfs_ioend_pool; > * in the future, too. > */ > enum { > - Opt_barrier, Opt_nobarrier, Opt_err > + Opt_barrier, Opt_nobarrier, Opt_inode64, Opt_err > }; > > static const match_table_t tokens = { > {Opt_barrier, "barrier"}, > {Opt_nobarrier, "nobarrier"}, > + {Opt_inode64, "inode64"}, > {Opt_err, NULL} > }; > > @@ -1043,6 +1044,7 @@ xfs_fs_remount( > > while ((p = strsep(&options, ",")) != NULL) { > int token; > + int i = 0; > > if (!*p) > continue; > @@ -1055,6 +1057,20 @@ xfs_fs_remount( > case Opt_nobarrier: > mp->m_flags &= ~XFS_MOUNT_BARRIER; > break; > + case Opt_inode64: > + > + for (i = 0; i < mp->m_sb.agcount; i++) { > + struct xfs_perag *pag; > + > + pag = xfs_perag_get(mp, index); > + pag->pagi_inodeok = 1; > + pag->pagf_metadata = 0; > + xfs_perag_put(pag); > + } > + mp->m_flags &= ~(XFS_MOUNT_32BITINODES | > + XFS_MOUNT_SMALL_INUMS); > + mp->m_maxagi = i; > + break; > default: > /* > * Logically we would return an error here to prevent > -- > 1.7.11.2 > > _______________________________________________ > xfs mailing list > xfs@oss.sgi.com > http://oss.sgi.com/mailman/listinfo/xfs -- --Carlos _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs