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.

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

[Encode] Fix ACM multi-channel AVC encode performance degradation #1897

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hoesheng
Copy link
Contributor

@hoesheng hoesheng commented Jan 9, 2025

disable DG2 Height alignment WA

@Sherry-Lin Sherry-Lin requested a review from bai-isaac January 15, 2025 06:45
@Sherry-Lin Sherry-Lin added Encode video encode related AVC AVC , MPEG2 VP8 etc. labels Jan 15, 2025
@@ -443,7 +443,6 @@ static bool InitTglMediaWaExt(struct GfxDeviceInfo *devInfo,
}

MEDIA_WR_WA(waTable, WaEnableVPPCopy, 1);
MEDIA_WR_WA(waTable, Wa_AvcUnalignedHeight, 1);
Copy link
Contributor

Choose a reason for hiding this comment

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

InitTglMediaWaExt is a common function for gen12, include TGL, DG2 ... please only remove for DG2, thanks.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated

@hoesheng hoesheng force-pushed the fix_acm_encode_perf branch from 0973462 to dbf4080 Compare January 16, 2025 11:24
disable DG2 Height alignment WA

Signed-off-by: Hoe, Sheng Yang <[email protected]>
@hoesheng hoesheng force-pushed the fix_acm_encode_perf branch from dbf4080 to f905a35 Compare January 16, 2025 11:26
@bai-isaac bai-isaac added the verifying PR: fix ready and verifying with build/test label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AVC AVC , MPEG2 VP8 etc. Encode video encode related verifying PR: fix ready and verifying with build/test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants