[PATCH][RESEND] pnfs/blocklayout: fix last_write_offset incorrectly set to page boundary

[PATCH][RESEND] pnfs/blocklayout: fix last_write_offset incorrectly set to page boundary

[Benjamin Coddington]

I'm resending this as the original was buried in a thread and I want to make
sure it gets picked up.  This fixes a bug that went in v4.8.

8<--------------------------------------------------------------------------

Commit 41963c10c47a35185e68cb9049f7a3493c94d2d7 sets the block layout's
last written byte to the offset of the end of the extent rather than the
end of the write which incorrectly updates the inode's size for
partial-page writes.

Fixes: 41963c10c47a ("pnfs/blocklayout: update last_write_offset atomically with extents")
Signed-off-by: Benjamin Coddington <bcodding@redhat.com>
---
 fs/nfs/blocklayout/blocklayout.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/fs/nfs/blocklayout/blocklayout.c b/fs/nfs/blocklayout/blocklayout.c
index 217847679f0e..2905479f214a 100644
--- a/fs/nfs/blocklayout/blocklayout.c
+++ b/fs/nfs/blocklayout/blocklayout.c
@@ -344,9 +344,10 @@ static void bl_write_cleanup(struct work_struct *work)
 		u64 start = hdr->args.offset & (loff_t)PAGE_MASK;
 		u64 end = (hdr->args.offset + hdr->args.count +
 			PAGE_SIZE - 1) & (loff_t)PAGE_MASK;
+		u64 lwb = hdr->args.offset + hdr->args.count;
 
 		ext_tree_mark_written(bl, start >> SECTOR_SHIFT,
-					(end - start) >> SECTOR_SHIFT, end);
+					(end - start) >> SECTOR_SHIFT, lwb);
 	}
 
 	pnfs_ld_write_done(hdr);
-- 
2.5.5

Re: [PATCH][RESEND] pnfs/blocklayout: fix last_write_offset incorrectly set to page boundary

[Christoph Hellwig]

On Tue, Oct 11, 2016 at 03:53:21PM -0400, Benjamin Coddington wrote:
> I'm resending this as the original was buried in a thread and I want to make
> sure it gets picked up.  This fixes a bug that went in v4.8.

Didn't I already give you my review for this?  Otherwise:

Reviewed-by: Christoph Hellwig <hch@lst.de>
Tested-by: Christoph Hellwig <hch@lst.de>

should also get a Cc to stable.

Re: [PATCH][RESEND] pnfs/blocklayout: fix last_write_offset incorrectly set to page boundary

[Anna Schumaker]

Hi Ben,

Thanks for the patch!  It didn't make it into my initial pull request for 4.9, but I am planning on sending it for an rc along with any bakeathon changes from next week.

I hope that's okay!
Anna

On 10/11/2016 04:17 PM, Christoph Hellwig wrote:
> On Tue, Oct 11, 2016 at 03:53:21PM -0400, Benjamin Coddington wrote:
>> I'm resending this as the original was buried in a thread and I want to make
>> sure it gets picked up.  This fixes a bug that went in v4.8.
> 
> Didn't I already give you my review for this?  Otherwise:
> 
> Reviewed-by: Christoph Hellwig <hch@lst.de>
> Tested-by: Christoph Hellwig <hch@lst.de>
> 
> should also get a Cc to stable.
> 

Re: [PATCH][RESEND] pnfs/blocklayout: fix last_write_offset incorrectly set to page boundary

[Christoph Hellwig]

On Thu, Oct 13, 2016 at 04:46:19PM -0400, Anna Schumaker wrote:
> Hi Ben,
> 
> Thanks for the patch!  It didn't make it into my initial pull request for 4.9, but I am planning on sending it for an rc along with any bakeathon changes from next week.

Any progress on that?  I'd really like to get the fix in before at least
-rc2.

Re: [PATCH][RESEND] pnfs/blocklayout: fix last_write_offset incorrectly set to page boundary

[Anna Schumaker]

Hi Christoph,

On 10/21/2016 08:18 AM, Christoph Hellwig wrote:
> On Thu, Oct 13, 2016 at 04:46:19PM -0400, Anna Schumaker wrote:
>> Hi Ben,
>>
>> Thanks for the patch!  It didn't make it into my initial pull request for 4.9, but I am planning on sending it for an rc along with any bakeathon changes from next week.
> 
> Any progress on that?  I'd really like to get the fix in before at least
> -rc2.
> 

I have the patch is my tree ready to go!  I've been holding back since the NFS Bakeathon is this week, and I was waiting to see if any bugfixes were posted that I could send off at the same time.

Anna