Skip to content
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

[WA for ChimoHMI] Use 64K memory alignment instead of 2MB alignment #125

Closed
wants to merge 14 commits into from
Closed

[WA for ChimoHMI] Use 64K memory alignment instead of 2MB alignment #125

wants to merge 14 commits into from

Conversation

ShenghuaLinINTEL
Copy link
Contributor

No description provided.

ShenghuaLinINTEL and others added 6 commits July 10, 2024 11:10
@ShenghuaLinINTEL @sysopenci
Revert "egl/android: Fix unsupported egl config r10g10b10a2, recordab…
f167e1d 
…le_android=0 for media cts"

This reverts commit 0ebc3b5.

Tracked-On: OAM-122074
Signed-off-by: Lin, Shenghua <[email protected]>
@ShenghuaLinINTEL @sysopenci
anv: Implement VK_EXT_device_memory_report
7f0f68e 
Vts checkVulkanDeviceMemoryReportSupport test requires that vulkan devices with
API version >= 1.1 must support VK_EXT_device_memory_report.

Tracked-On: OAM-122094
Signed-off-by: Huang Rui <[email protected]>
Signed-off-by: shenghualin <[email protected]>
@ShenghuaLinINTEL @sysopenci
Revert "INTERNAL: Revert 530ae32:iris:Fix to release BO immediately i…
2a921bb 
…f not busy"

This reverts commit de912b6.
This can avoid CTS incorrectly report "memory leak" if release BO immediately

Tracked-On: OAM-122096
Signed-off-by: Lin, Shenghua <[email protected]>
@phreer @sysopenci
anv: Add idep_genxml to dependencies of grl_genX_libs
71d0c8d 
Fix race condition when building ANV:

    In file included from src/intel/vulkan/grl/grl_metakernel_build_primref.c:40:
    ../src/intel/common/mi_builder.h:28:10: fatal error: 'genxml/genX_bits.h' file not found
    #include "genxml/genX_bits.h"

This change is already included in commit
41b2ed6 (genxml: generate opencl
packing headers) in the upstream.  Hence when updating mesa version,
this change can be safely removed.

Test-done: build mesa multiple times

Tracked-On: OAM-122542
Signed-off-by: Weifeng Liu <[email protected]>
@ShenghuaLinINTEL @sysopenci
INTERNAL/CI: Delete .gitlab-ci to avoid Github security scan issue
7a93157 
Tracked-On: OAM-122380
Signed-off-by: Lin, Shenghua <[email protected]>
@sysopenci
Add legacy dataspace value supporting
e68b024 
In case of legacy dataspace value is passed down by
framework/av/media code.

Tracked-On: OAM-122156
Signed-off-by: ZhuChenyanX <[email protected]>
@sysopenci
Copy link

Expected syntax:
Commit message one-line summary

  Extra paragraphs can be added here.

  Longer description that explains the problem / solution
  or whatever that needs to be said

  It can contain
    multiple lines
    multiple paragraphs
    blank lines
    references

  But each paragraph must be followed by a blank line

  However, you need to keep the 'Tracked-On', 'Change-Id',
  'Signed-off-by', ..., fields in the **lastest** paragraph of
  the commit message.

  Change-Id: I9e58f79c7faf7f0edb9792e8b5322ab161331141
  Tracked-On: https://jira.server.ext/browse/PROJECTA-123
  Signed-off-by: Developer Name <[email protected]>

@sysopenci sysopenci added Pending Developer Approval Pending Developer Approval Pending PR Review Pending PR Review Engineering Build Not Started Engineering Build Not Started labels Jul 24, 2024
@sysopenci
Copy link

Improper Commit Message
Valid Commit Message
Improper Jira Status,
jira status not in ['In Progress','Implemented']

@sysopenci
Copy link

Expected syntax:
Commit message one-line summary

  Extra paragraphs can be added here.

  Longer description that explains the problem / solution
  or whatever that needs to be said

  It can contain
    multiple lines
    multiple paragraphs
    blank lines
    references

  But each paragraph must be followed by a blank line

  However, you need to keep the 'Tracked-On', 'Change-Id',
  'Signed-off-by', ..., fields in the **lastest** paragraph of
  the commit message.

  Change-Id: I9e58f79c7faf7f0edb9792e8b5322ab161331141
  Tracked-On: https://jira.server.ext/browse/PROJECTA-123
  Signed-off-by: Developer Name <[email protected]>

@sysopenci
Copy link

Expected syntax:
Commit message one-line summary

  Extra paragraphs can be added here.

  Longer description that explains the problem / solution
  or whatever that needs to be said

  It can contain
    multiple lines
    multiple paragraphs
    blank lines
    references

  But each paragraph must be followed by a blank line

  However, you need to keep the 'Tracked-On', 'Change-Id',
  'Signed-off-by', ..., fields in the **lastest** paragraph of
  the commit message.

  Change-Id: I9e58f79c7faf7f0edb9792e8b5322ab161331141
  Tracked-On: https://jira.server.ext/browse/PROJECTA-123
  Signed-off-by: Developer Name <[email protected]>

@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

zmike and others added 6 commits July 25, 2024 21:52
@zmike @sysopenci
frontends/dri: only release pipe when screen init fails
b1e8d3d 
the caller (driCreateNewScreen3) will always call dri_destroy_screen()
when these functions return failure, so releasing the screen
is always wrong

cc: mesa-stable

Tracked-On: OAM-122605
Reviewed-by: Adam Jackson <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/29021>
Signed-off-by: Weifeng Liu <[email protected]>
Signed-off-by: Mike Blumenkrantz <[email protected]>
@zmike @sysopenci
frontends/dri: always init opencl_func_mutex in InitScreen hooks
b95a14b 
this otherwise leads to a mismatch where some types of screen may have
the mutex initialized while others don't, in which case dri_release_screen()
will attempt to destroy an uninitialized mutex

cc: mesa-stable

Tracked-On: OAM-122605
Reviewed-by: Adam Jackson <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/29021>
Signed-off-by: Weifeng Liu <[email protected]>
Signed-off-by: Mike Blumenkrantz <[email protected]>
@iViggyPrabhu
Updated CI workflow
057e750
@yhe39 @sysopenci
Refine the dGPU render
44ed50f 
Check the dGPU through whether has local memory.
Calculate the i915 render node number for better
compatibility.
The GLES open one available render node, vulkan
open all available render nodes.
Enalbe the vulkan dGPU render.

Tracked-On: OAM-122333
Signed-off-by: HeYue <[email protected]>
@phreer @sysopenci
Make Android specific code specific to Android
c275c62 
We have some code specific to Android only and it should be built only
on Android so that this code base could possibly be built on GNU/Linux
systems.

Also replace \xC2\xA0 (non-breaking space) with normal space for
consistence.

Signed-off-by: Weifeng Liu <[email protected]>
@phreer @sysopenci
android: Open dGPU for composition in surfaceflinger
97324f2 
When we have both dGPU and iGPU available, prefer dGPU for compositing
because dGPU can cover more cases for us, e.g., allocating buffers in
device memory.

As a side work, also format the surrounding code for consistency.

Tracked-On: OAM-124060
Signed-off-by: Weifeng Liu <[email protected]>
@sysopenci sysopenci added the Stale Stale label for inactive open prs label Sep 6, 2024
@llandwerlin-intel @sysopenci
anv: explicitly disable BT pool allocations at device init
1588bfe 
Ported from upstream:
https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/30841?commit_id=1f9c40a8d1b2ec3702862371af3a934e9a0c9cda

The default state doesn't seem well defined (or kernel driver bug
maybe?). Let's just set it to disabled on platforms where we're not
using it.

Signed-off-by: Lionel Landwerlin <[email protected]>
Cc: mesa-stable
Found-by: Chuansheng Liu <[email protected]>
Reviewed-by: Tapani Pälli <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/30841>

Tracked-On: OAM-124615
Signed-off-by: Lin, Shenghua <[email protected]>
@ShenghuaLinINTEL ShenghuaLinINTEL changed the title [WA] Use 64K memory alignment instead of 2MB alignment [Draft] Use 64K memory alignment instead of 2MB alignment Sep 25, 2024
@ShenghuaLinINTEL ShenghuaLinINTEL changed the title [Draft] Use 64K memory alignment instead of 2MB alignment [Draft] [WA for ChimoHMI] Use 64K memory alignment instead of 2MB alignment Sep 25, 2024
@ShenghuaLinINTEL
[WA] Use 64KB alignment instead of 2MB alignment
955c355 
Chimo app allocate descriptor pool so frequently, with 2MB alignment
the 2Gb descriptor pool heap vm address space is used up quickly.
@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

@ShenghuaLinINTEL ShenghuaLinINTEL changed the title [Draft] [WA for ChimoHMI] Use 64K memory alignment instead of 2MB alignment [WA for ChimoHMI] Use 64K memory alignment instead of 2MB alignment Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@feijiang1 feijiang1 Awaiting requested review from feijiang1

Assignees
No one assigned
Labels
Engineering Build Not Started Engineering Build Not Started Invalid commit message Pending Developer Approval Pending Developer Approval Pending PR Review Pending PR Review Stale Stale label for inactive open prs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@ShenghuaLinINTEL @sysopenci @phreer @zmike @iViggyPrabhu @yhe39 @llandwerlin-intel