Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 12 Aug 2006 12:03:28 +0200
From:      Pawel Jakub Dawidek <pjd@FreeBSD.org>
To:        Robert Watson <rwatson@FreeBSD.org>
Cc:        freebsd-fs@freebsd.org, freebsd-current@freebsd.org, Sean Bryant <bryants@gmail.com>, freebsd-geom@freebsd.org
Subject:   Re: GJournal (hopefully) final patches.
Message-ID:  <20060812100328.GB73241@garage.freebsd.pl>
In-Reply-To: <20060811143257.E2260@fledge.watson.org>
References:  <20060808195202.GA1564@garage.freebsd.pl> <f6791cc60608101418r52c42aa0mad896951db5b75a@mail.gmail.com> <f6791cc60608101554w2b10d8a5y9dc87f2198080281@mail.gmail.com> <20060810232255.GE1389@garage.freebsd.pl> <20060811143257.E2260@fledge.watson.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--aM3YZ0Iwxop3KEKx
Content-Type: text/plain; charset=iso-8859-2
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Fri, Aug 11, 2006 at 02:33:44PM +0100, Robert Watson wrote:
>=20
> On Fri, 11 Aug 2006, Pawel Jakub Dawidek wrote:
>=20
> >2. Confirm that partition with your file system is _not_ 4 sectors
> >  aligned. If it is not 4 sectors aligned, it is safe for gjournal to
> >  use its last sector for metadata. You can verify this by running:
> >
> >	test `diskinfo /dev/<data_partition> | awk '{print $4}' | xargs -J X ec=
ho X % 4 | bc` -eq 0 && echo ok || echo not ok
> >
> >Then you need to run those commands:
> >
> >	# gjournal label /dev/<data_partition> /dev/<journal_partition>
> >	# tunefs -n disable -J enable /dev/<data_partition>.journal
>=20
> Sounds like a "gjournal testalignment /dev/partition" would be a very use=
ful thing.  Or even just doing it as part of gjournal label?

Sounds fair. I integrated those checks into 'label' subcommand:

	When configuring journaling on one provider only, check if UFS
	file system exists there and if it exists demand -f flag before
	destroying it.
	When configuring journaling on two providers, check if UFS
	exists on the data provider and if the last sector is used by
	the file system. If it is used, demand -f flag before
	overwritting it.

Next, I want to think a bit how to make those checks more general, so
they can be used also in other classes.

--=20
Pawel Jakub Dawidek                       http://www.wheel.pl
pjd@FreeBSD.org                           http://www.FreeBSD.org
FreeBSD committer                         Am I Evil? Yes, I Am!

--aM3YZ0Iwxop3KEKx
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.4 (FreeBSD)

iD8DBQFE3adwForvXbEpPzQRArQjAKCG2gX0ac4N+mdVe13Bo/IJbAILgwCgka8D
Ndh04x6cz/C/D7L8kfURqeo=
=jYpF
-----END PGP SIGNATURE-----

--aM3YZ0Iwxop3KEKx--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060812100328.GB73241>