On Wed, Mar 6, 2024 at 12:24 PM Luis Henriques <lhenriques@xxxxxxx> wrote: > > xiubli@xxxxxxxxxx writes: > > > From: Xiubo Li <xiubli@xxxxxxxxxx> > > > > The osd code has remove cursor initilizing code and this will make > > the sparse read state into a infinite loop. We should initialize > > the cursor just before each sparse-read in messnger v2. > > > > Cc: stable@xxxxxxxxxxxxxxx > > URL: https://tracker.ceph.com/issues/64607 > > Fixes: 8e46a2d068c9 ("libceph: just wait for more data to be available on the socket") > > Reported-by: Luis Henriques <lhenriques@xxxxxxx> > > Signed-off-by: Xiubo Li <xiubli@xxxxxxxxxx> > > --- > > > > V2: > > - Just removed the unnecessary 'sparse_read_total' check. > > > > Thanks a lot for the quick fix, Xiubo. FWIW: > > Tested-by: Luis Henriques <lhenriques@xxxxxxx> Thank you for catching this, Luis! I'm still lacking clarity on how this got missed, but hopefully the fs suite will improve with regard to fscrypt + ms_type coverage. I have staged the fix with a minor tweak to use msg local variable instead of con->in_msg and reworded changelog: https://github.com/ceph/ceph-client/commit/321e3c3de53c7530cd518219d01f04e7e32a9d23 Ilya
