xfs
[Top] [All Lists]

Re: [PATCH v2 1/3] xfs: use b_maps[] for discontiguous buffers

To: Mark Tinguely <tinguely@xxxxxxx>
Subject: Re: [PATCH v2 1/3] xfs: use b_maps[] for discontiguous buffers
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Mon, 3 Dec 2012 10:31:28 +1100
Cc: xfs@xxxxxxxxxxx
In-reply-to: <20121128222622.688093599@xxxxxxx>
References: <20121128222309.109033307@.sgi.com> <20121128222622.688093599@xxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Wed, Nov 28, 2012 at 04:23:10PM -0600, Mark Tinguely wrote:
> Commits starting at 77c1a08 introduced a multiple segment support
> to xfs_buf. xfs_trans_buf_item_match() could not find a multi-segment
> buffer in the transaction because it was looking at the single segment
> block number rather than the multi-segment b_maps[0].bm.bn. This
> results on a recursive buffer lock that can never be satisfied.
> 
> This patch:
>  1) Changed the remaining b_map accesses to be b_maps[0] accesses.
>  2) Renames the single segment b_map structure to __b_map to avoid
>     future confusion.
> 
> Signed-off-by: Mark Tinguely <tinguely@xxxxxxx>

Looks good.

Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
-- 
Dave Chinner
david@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>
  • Re: [PATCH v2 1/3] xfs: use b_maps[] for discontiguous buffers, Dave Chinner <=