aboutsummaryrefslogtreecommitdiff
path: root/src/capture/xcomposite.c
diff options
context:
space:
mode:
authordec05eba <dec05eba@protonmail.com>2024-06-19 00:05:23 +0200
committerdec05eba <dec05eba@protonmail.com>2024-06-19 00:05:23 +0200
commitb3acabbf34aa3638def53abc30bff3ef646cd60a (patch)
tree5320c4e8a9dca0e33292afb87e56423f606b8a2a /src/capture/xcomposite.c
parent3e2e2444d9fa65ae404604542689cf7829c99cfa (diff)
AMD: align av1 to extra padding because of hardware bug
Diffstat (limited to 'src/capture/xcomposite.c')
-rw-r--r--src/capture/xcomposite.c10
1 files changed, 10 insertions, 0 deletions
diff --git a/src/capture/xcomposite.c b/src/capture/xcomposite.c
index 236cfc0..59df070 100644
--- a/src/capture/xcomposite.c
+++ b/src/capture/xcomposite.c
@@ -107,6 +107,16 @@ int gsr_capture_xcomposite_start(gsr_capture_xcomposite *self, AVCodecContext *v
// TODO: dont do this if using ffmpeg reports that this is not needed (AMD driver bug that was fixed recently)
video_codec_context->width = FFALIGN(video_size.x, 64);
video_codec_context->height = FFALIGN(video_size.y, 16);
+ } else if(self->params.egl->gpu_info.vendor == GSR_GPU_VENDOR_AMD && video_codec_context->codec_id == AV_CODEC_ID_AV1) {
+ // TODO: Dont do this for VCN 5 and forward which should fix this hardware bug
+ video_codec_context->width = FFALIGN(video_size.x, 64);
+ // AMD driver has special case handling for 1080 height to set it to 1082 instead of 1088 (1080 aligned to 16).
+ // TODO: Set height to 1082 in this case, but it wont work because it will be aligned to 1088.
+ if(video_size.y == 1080) {
+ video_codec_context->height = 1080;
+ } else {
+ video_codec_context->height = FFALIGN(video_size.y, 16);
+ }
} else {
video_codec_context->width = FFALIGN(video_size.x, 2);
video_codec_context->height = FFALIGN(video_size.y, 2);