Skip to content

squid: blk/kerneldevice: notify_all only required when discard_drain wait for condition#62151

Merged
SrinivasaBharath merged 1 commit intoceph:squidfrom
YiteGu:wip-70148-squid
Apr 15, 2025
Merged

squid: blk/kerneldevice: notify_all only required when discard_drain wait for condition#62151
SrinivasaBharath merged 1 commit intoceph:squidfrom
YiteGu:wip-70148-squid

Conversation

@YiteGu
Copy link
Member

@YiteGu YiteGu commented Mar 6, 2025

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


backport of #59529
parent tracker: https://tracker.ceph.com/issues/67835

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

@YiteGu YiteGu requested a review from a team as a code owner March 6, 2025 09:27
@YiteGu YiteGu added this to the squid milestone Mar 6, 2025
@github-actions github-actions bot added the core label Mar 6, 2025
@YiteGu YiteGu requested a review from ifed01 March 6, 2025 09:28
@YiteGu
Copy link
Member Author

YiteGu commented Mar 7, 2025

jenkins test api

@NitzanMordhai
Copy link
Contributor

@NitzanMordhai
Copy link
Contributor

jenkins test docs

@NitzanMordhai
Copy link
Contributor

@YiteGu it looks like you need to rebase before we can merge

…r condition

This will cause discard threads to wake up each other
if bdev_async_discard_threads >= 2.

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

Signed-off-by: Yite Gu <yitegu0@gmail.com>
(cherry picked from commit cec2e45)
@YiteGu YiteGu force-pushed the wip-70148-squid branch from 3205100 to ce3906e Compare April 9, 2025 08:15
@YiteGu
Copy link
Member Author

YiteGu commented Apr 9, 2025

@YiteGu it looks like you need to rebase before we can merge

done, it no conflict after rebase.

@YiteGu
Copy link
Member Author

YiteGu commented Apr 10, 2025

Hi @ljflores, pls help handle this docs check fail.

@SrinivasaBharath SrinivasaBharath merged commit bdaad91 into ceph:squid Apr 15, 2025
9 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