Skip to content

Bug fixes ported from u/mr0/master branch #207

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
Open
phreer wants to merge 2 commits into
base: celadon/u/mr0/stable
Choose a base branch
from
Open

Bug fixes ported from u/mr0/master branch #207

phreer wants to merge 2 commits into from

Conversation

@phreer
Copy link

@phreer phreer commented Mar 25, 2025

No description provided.

phreer added 2 commits March 25, 2025 14:19
@phreer
Fix double free in driver loading
ba7f797 
Tracked-On: OAM-131321
Signed-off-by: Weifeng Liu <weifeng.liu@intel.com>
@phreer
Fix bugs in driver loading
4b38f78 
* Reserve enough space for storing drm fds.
* Don't close fd when we want to destroy a driver and recreate it soon.

Tracked-On: OAM-131321
Signed-off-by: Weifeng Liu <weifeng.liu@intel.com>
@phreer phreer changed the title U mr0 stable/fix memory bugs Bug fixes Mar 25, 2025
@sysopenci sysopenci added Pending Developer Approval Pending Developer Approval Pending PR Review Pending PR Review Valid commit message Engineering Build Not Started Engineering Build Not Started labels Mar 25, 2025
@phreer phreer changed the title Bug fixes Bug fixes ported from u/mr0/master branch Mar 25, 2025
@sysopenci sysopenci added the Pending Integration Approval Pending Integration Approval label Mar 25, 2025
feijiang1
feijiang1 approved these changes Mar 26, 2025
View reviewed changes
Copy link

@feijiang1 feijiang1 left a comment

Choose a reason for hiding this comment

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

LGTM

@sysopenci sysopenci added PR approved and removed Pending PR Review Pending PR Review labels Mar 26, 2025
@sysopenci sysopenci added Developer Approved and removed Pending Developer Approval Pending Developer Approval labels Mar 26, 2025
@sysopenci
Copy link

sysopenci commented Mar 26, 2025

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

sysopenci commented Mar 26, 2025

FAILURE: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added Pending Developer Approval Pending Developer Approval Pending PR Review Pending PR Review Engineering Build Not Started Engineering Build Not Started Pending Integration Approval Pending Integration Approval and removed Developer Approved Pending Integration Approval Pending Integration Approval Engineering Build Failed labels Apr 2, 2025
@sysopenci
Copy link

sysopenci commented Apr 2, 2025

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

sysopenci commented Apr 2, 2025

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

sysopenci commented Apr 2, 2025

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added Engineering Build Successful Engineering Build Successful Developer Approved and removed Engineering Build Not Started Engineering Build Not Started Pending Developer Approval Pending Developer Approval labels Apr 2, 2025
@sysopenci
Copy link

sysopenci commented Apr 2, 2025

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

feijiang1
feijiang1 approved these changes Apr 2, 2025
View reviewed changes
Copy link

@feijiang1 feijiang1 left a comment

Choose a reason for hiding this comment

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

LGTM

@sysopenci sysopenci added PR approved and removed Pending PR Review Pending PR Review labels Apr 2, 2025
feijiang1
feijiang1 approved these changes Apr 17, 2025
View reviewed changes
Copy link

@feijiang1 feijiang1 left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@feijiang1 feijiang1 feijiang1 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Developer Approved Engineering Build Successful Engineering Build Successful Pending Integration Approval Pending Integration Approval PR approved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@phreer @sysopenci @feijiang1