From owner-freebsd-fs@FreeBSD.ORG Thu Aug 4 15:40:10 2011 Return-Path: Delivered-To: freebsd-fs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 9B7CA1065670 for ; Thu, 4 Aug 2011 15:40:10 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 860F38FC13 for ; Thu, 4 Aug 2011 15:40:10 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.4/8.14.4) with ESMTP id p74FeA9b072230 for ; Thu, 4 Aug 2011 15:40:10 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.4/8.14.4/Submit) id p74FeALk072229; Thu, 4 Aug 2011 15:40:10 GMT (envelope-from gnats) Date: Thu, 4 Aug 2011 15:40:10 GMT Message-Id: <201108041540.p74FeALk072229@freefall.freebsd.org> To: freebsd-fs@FreeBSD.org From: Borja Marcos Cc: Subject: Re: kern/157728: [zfs] zfs (v28) incremental receive may leave behind temporary clones X-BeenThere: freebsd-fs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Borja Marcos List-Id: Filesystems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 04 Aug 2011 15:40:10 -0000 The following reply was made to PR kern/157728; it has been noted by GNATS. From: Borja Marcos To: Martin Matuska Cc: bug-followup@FreeBSD.org, Pawel Jakub Dawidek Subject: Re: kern/157728: [zfs] zfs (v28) incremental receive may leave behind temporary clones Date: Thu, 4 Aug 2011 17:37:26 +0200 On Aug 4, 2011, at 4:18 PM, Martin Matuska wrote: > But I still think we don't have to prefetch data we are not = processing. >=20 > I don't think that there will be any ugly interaction in this case. > The idea of the prefetch code is to speed up access to the data > structure by caching it into memory. > So what we don't prefetch (is not cached) will be read the normal way > (and not from cache). >=20 > If you follow its history, you can see it well: >=20 > Prefetch for zfs list was introduced in OpenSolaris changeset 8415 and > didn't change very much since that point: > http://hg.openindiana.org/illumos-gate/rev/d5525cd1cbc2 >=20 > If you remove that code, it will still work the way it should, but = slower :) > I still see no problem in not-prefetching hidden datasets. Understood :) Thank you very much. As I said, I'm not that familiar with = the internals. I'm going to try the patch and will let you know the outcome. I guess = it will effectively fix it. Best regards, Borja.