From mboxrd@z Thu Jan  1 00:00:00 1970
From: Alberto Bertogli <albertito@blitiri.com.ar>
Subject: Re: [RFC PATCH] bio-integrity: Copy bip_buf and bip_size in
	bio_integrity_clone()
Date: Sun, 24 May 2009 04:16:17 -0300
Message-ID: <20090524071617.GH1376@blitiri.com.ar>
References: <20090524042035.GF1376@blitiri.com.ar>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: linux-fsdevel@vger.kernel.org, Jens Axboe <jens.axboe@oracle.com>,
	"Martin K. Petersen" <martin.petersen@oracle.com>
To: linux-kernel@vger.kernel.org
Return-path: <linux-fsdevel-owner@vger.kernel.org>
Received: from alerce.vps.bitfolk.com ([212.13.194.134]:3619 "EHLO
	alerce.vps.bitfolk.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1754390AbZEXHTi (ORCPT
	<rfc822;linux-fsdevel@vger.kernel.org>);
	Sun, 24 May 2009 03:19:38 -0400
Content-Disposition: inline
In-Reply-To: <20090524042035.GF1376@blitiri.com.ar>
Sender: linux-fsdevel-owner@vger.kernel.org
List-ID: <linux-fsdevel.vger.kernel.org>

On Sun, May 24, 2009 at 01:20:35AM -0300, Alberto Bertogli wrote:
> While at it, I found that bio_integrity_clone() does not clone neither bip_buf
> nor bip_size, which already copies the bvec, which should have the same data
> because it's allocated in bio_integrity_prep().

In case that makes sense, updating bip_buf and bip_size (if bip_buf is not
NULL) in bio_integrity_trim() does too, doesn't it?

Thanks a lot,
		Alberto