Skip to content

[Thread Data] do not reserve string data in thread #61278

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
kangguangli merged 1 commit into from
Jan 29, 2024
Merged

[Thread Data] do not reserve string data in thread #61278

kangguangli merged 1 commit into from
Jan 29, 2024

Conversation

@kangguangli
Copy link
Contributor

@kangguangli kangguangli commented Jan 29, 2024 •
edited
Loading

PR types

Others

PR changes

Others

Description

do not reserve string data in thread, avoid following log:

I0129 11:21:01.220558  5208 pir_interpreter.cc:1746] 
done: RunInstructionBase OP id:23 name:pd_op.fused_conv2d_add_act type:kGpuAsync runs on MainThreadDeviceKernelLaunch_thread_0HostTasks_thread_0DeviceKernelLaunch_thread_0HostTasks_thread_0DeviceKernelLaunch_thread_0HostTasks_thread_0DeviceKernelLaunch_thread_0HostTasks_thread_0DeviceKernelLaunch_thread_0HostTasks_thread_0DeviceKernelLaunch_thread_0HostTasks_thread_0DeviceKernelLaunch_thread_0HostTasks_thread_0DeviceKernelLaunch_thread_0HostTasks_thread_0DeviceKernelLaunch_thread_0HostTasks_thread_0DeviceKernelLaunch_thread_0HostTasks_thread_0...
After: Place(gpu:0) Op(pd_op.fused_conv2d_add_act), inputs:{...}, outputs:{...}.

Others

Pcard-67164

@kangguangli kangguangli requested a review from From00 January 29, 2024 06:50
From00
From00 approved these changes Jan 29, 2024
View reviewed changes
Copy link
Contributor

@From00 From00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kangguangli kangguangli merged commit a9d8911 into PaddlePaddle:develop Jan 29, 2024
@kangguangli kangguangli deleted the donot_reserve_string_data_in_thread branch January 29, 2024 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@From00 From00 From00 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kangguangli @From00