Re: [PATCH] ceph: fix missing snapshot context in write operations

[Viacheslav Dubeyko <Slava.Dubeyko@ibm.com>]

On Wed, 2025-09-24 at 17:58 +0800, ethanwu wrote:
> This series addresses two instances where Ceph filesystem operations
> were missing proper snapshot context handling, which could lead to
> data inconsistencies in snapshots.
> 

It doesn't look like series. You can prepare patch set by using command:

git format-patch --cover-letter -2 HEAD

Thanks,
Slava.

> The issue occurs in two scenarios:
> 1. ceph_zero_partial_object() during fallocate punch hole operations
> 2. ceph_uninline_data() when converting inline data to regular objects
> 
> Both functions were passing NULL snapshot context to OSD write operations
> instead of acquiring the appropriate context from either pending cap snaps
> or the inode's head snapc. This could result in snapshot data corruption
> where subsequent reads from snapshots would return modified data instead
> of the original snapshot content.
> 
> The fix ensures that proper snapshot context is acquired and passed to
> all OSD write operations in these code paths.
> 
> ethanwu (2):
>   ceph: fix snapshot context missing in ceph_zero_partial_object
>   ceph: fix snapshot context missing in ceph_uninline_data
> 
>  fs/ceph/addr.c | 19 +++++++++++++++++--
>  fs/ceph/file.c | 17 ++++++++++++++++-
>  2 files changed, 33 insertions(+), 3 deletions(-)