Skip to content

ITS: GPU: partial revert of memory clearing#15023

Merged
shahor02 merged 2 commits intoAliceO2Group:devfrom
f3sch:its/gpu/fix2
Feb 6, 2026
Merged

ITS: GPU: partial revert of memory clearing#15023
shahor02 merged 2 commits intoAliceO2Group:devfrom
f3sch:its/gpu/fix2

Conversation

@f3sch
Copy link
Collaborator

@f3sch f3sch commented Feb 5, 2026

This partially reverts the previous commit (which introduced some kind of race condition) but keeps the added memory clearing of the function itself.

f3sch added 2 commits February 5, 2026 17:46
This reverts commit aa3ef37.

Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@f3sch f3sch marked this pull request as ready for review February 5, 2026 16:50
@f3sch
Copy link
Collaborator Author

f3sch commented Feb 6, 2026

@shahor02 please merge so the GPU code is not left in a broken state :) thanks!

@shahor02
Copy link
Collaborator

shahor02 commented Feb 6, 2026

Keeping separate commits.

@shahor02 shahor02 merged commit 19803f6 into AliceO2Group:dev Feb 6, 2026
11 of 12 checks passed
@f3sch f3sch deleted the its/gpu/fix2 branch February 6, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants