Skip to content

tentacle: rgw/d3n: fix valgrind invalid read during exit#63436

Merged
yuriw merged 1 commit intoceph:tentaclefrom
mkogan1:wip-71420-tentacle
Jul 3, 2025
Merged

tentacle: rgw/d3n: fix valgrind invalid read during exit#63436
yuriw merged 1 commit intoceph:tentaclefrom
mkogan1:wip-71420-tentacle

Conversation

@mkogan1
Copy link
Copy Markdown
Contributor

@mkogan1 mkogan1 commented May 22, 2025

backport tracker: https://tracker.ceph.com/issues/71420


backport of #63352
parent tracker: https://tracker.ceph.com/issues/64835

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

The valgrind indication is a false positive in this case, is seems there is a race of memory read and free of the libaio and aio kernel threads which had not timer-d out yet during exit()

reducing the aio_idle_time timeout to mitigate

```
man aio_init
...
aio_idle_time
This field specifies the amount of time in seconds that a worker thread
should wait for further requests before terminating, after having
completed a previous request.   The
default value is 1.
...
```

Fixes: https://tracker.ceph.com/issues/64835

Signed-off-by: Mark Kogan <mkogan@ibm.com>
(cherry picked from commit 31fae3a)
@mkogan1 mkogan1 requested a review from a team as a code owner May 22, 2025 16:51
@mkogan1 mkogan1 added this to the tentacle milestone May 22, 2025
@mkogan1 mkogan1 requested a review from cbodley May 22, 2025 16:53
@adamemerson
Copy link
Copy Markdown
Contributor

jenkins test windows

@yuriw yuriw merged commit bd64382 into ceph:tentacle Jul 3, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants