Skip to content

Conversation

@smithdh
Copy link
Contributor

@smithdh smithdh commented Aug 21, 2025

No description provided.

@abh3
Copy link
Member

abh3 commented Aug 21, 2025

How is that? It obviously breaks internal linkage but I could not find any public dependence on ClChannel remaining unchanged.

@smithdh smithdh force-pushed the ds_forcedisc_deadlock branch from c256e6f to 7eac856 Compare August 22, 2025 08:53
@amadio
Copy link
Member

amadio commented Aug 22, 2025

@abh3 You are indeed right, the Channel implementation is not exposed, so the change passed the ABI checks. This can safely go into 5.9.

@smithdh
Copy link
Contributor Author

smithdh commented Aug 26, 2025

I found this still needs some work; I'll have to keep in in draft for a while longer.

@amadio amadio force-pushed the devel branch 2 times, most recently from a934415 to 1122d21 Compare September 8, 2025 17:54
@smithdh smithdh force-pushed the ds_forcedisc_deadlock branch from 7eac856 to 8c9a2a3 Compare September 9, 2025 09:51
@amadio
Copy link
Member

amadio commented Sep 9, 2025

Nevermind, I see this is based on devel... Could you please rebase on master?

@smithdh smithdh force-pushed the ds_forcedisc_deadlock branch from 8c9a2a3 to b3b6b66 Compare September 9, 2025 12:56
@smithdh
Copy link
Contributor Author

smithdh commented Sep 9, 2025

hi @amadio ok, will rebase to master, just a moment.

@smithdh smithdh force-pushed the ds_forcedisc_deadlock branch from b3b6b66 to ad93707 Compare September 9, 2025 13:26
@smithdh smithdh changed the base branch from devel to master September 9, 2025 13:27
@amadio
Copy link
Member

amadio commented Sep 9, 2025

@smithdh Thank you! Please let me know when you are done, and take the pull request out of draft, so I can include it for 5.9. Cheers,

@amadio amadio added this to the 5.9.0 milestone Sep 9, 2025
@smithdh smithdh marked this pull request as ready for review September 12, 2025 08:50
@smithdh smithdh force-pushed the ds_forcedisc_deadlock branch from ad93707 to 0efe3a1 Compare September 12, 2025 08:51
@amadio amadio linked an issue Sep 12, 2025 that may be closed by this pull request
@amadio amadio merged commit 147d003 into xrootd:master Sep 12, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Deadlock around idle timeout & close of stream with multiple substreams

3 participants