Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 8 Sep 2010 11:48:55 +0300
From:      Kostik Belousov <kostikbel@gmail.com>
To:        Andriy Gapon <avg@freebsd.org>
Cc:        freebsd-fs@freebsd.org, Pawel Jakub Dawidek <pjd@freebsd.org>
Subject:   Re: zfs very poor performance compared to ufs due to lack of cache?
Message-ID:  <20100908084855.GF2465@deviant.kiev.zoral.com.ua>
In-Reply-To: <4C873914.40404@freebsd.org>
References:  <5DB6E7C798E44D33A05673F4B773405E@multiplay.co.uk> <AANLkTikNhsj5myhQCoPaNytUbpHtox1vg9AZm1N-OcMO@mail.gmail.com> <4C85E91E.1010602@icyb.net.ua> <4C873914.40404@freebsd.org>

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

--E7i4zwmWs5DOuDSH
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Sep 08, 2010 at 10:19:48AM +0300, Andriy Gapon wrote:
> on 07/09/2010 10:26 Andriy Gapon said the following:
> > Interesting.  I briefly looked at the code in mappedread(), zfs_vnops.c=
, and I
> > have a VM question.
> > Shouldn't we mark the corresponding page bits as valid after reading da=
ta into
> > the page?
> > I specifically speak of the block that starts with the following line:
> > } else if (m !=3D NULL && uio->uio_segflg =3D=3D UIO_NOCOPY) {
> > I am taking mdstart_swap as an example and it does m->valid =3D VM_PAGE=
_BITS_ALL.
> >=20
>=20
> I've chatted with and conclusion seems to be that vm_page_set_validclean(=
) call
> should be added at the end of the block.
>=20
> Perhaps, something like this:
>=20
> --- a/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c
> +++ b/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c
> @@ -500,6 +500,7 @@ again:
>  				sched_unpin();
>  			}
>  			VM_OBJECT_LOCK(obj);
> +			vm_page_set_validclean(m, off, bytes);
Only if error =3D=3D 0, perhaps ?

>  			vm_page_wakeup(m);
>  			if (error =3D=3D 0)
>  				uio->uio_resid -=3D bytes;
>=20
>=20
> BTW, I think that 'off' variable can be made of 'int' type, as it is an o=
ffset
> within a page; no need for int64.
>=20
> --=20
> Andriy Gapon

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

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

iEYEARECAAYFAkyHTfcACgkQC3+MBN1Mb4gIeACgm4qHkuJVwqfQNcsYuzgpSkcQ
tvoAoO0JAwtgqTdovoyGUmYgX6UDJaPO
=GcfI
-----END PGP SIGNATURE-----

--E7i4zwmWs5DOuDSH--



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