On Mon, Apr 21, 2008 at 05:29:12PM +0200, Miklos Szeredi wrote: > This instead? > > if (offset > 0) { > ret = 0; > map_bh(bh_result, inode->i_sb, offset); > if (remain > max_blocks) > remain = max_blocks; > bh_result->b_size = (remain << inode->i_blkbits); > goto out_brelse; > } Much cleaner, thanks! -- Bob Copeland %% www.bobcopeland.com -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html