Quoting Xiang, Haihao (2023-11-24 07:17:36) > From: Haihao Xiang <haihao.xi...@intel.com> > > It is possible that renderD128 doesn't exist but renderD129 is > available in a system (see [1]). This change can make sure the default > DRM device selection works even if renderD128 doesn't exist. > > [1] > https://github.com/intel/intel-device-plugins-for-kubernetes/blob/main/cmd/gpu_plugin/README.md#issues-with-media-workloads-on-multi-gpu-setups > > Signed-off-by: Haihao Xiang <haihao.xi...@intel.com> > --- > libavutil/hwcontext_vaapi.c | 14 ++++++++++++-- > 1 file changed, 12 insertions(+), 2 deletions(-) > > diff --git a/libavutil/hwcontext_vaapi.c b/libavutil/hwcontext_vaapi.c > index 12bc95119a..6d542a9bc9 100644 > --- a/libavutil/hwcontext_vaapi.c > +++ b/libavutil/hwcontext_vaapi.c > @@ -1733,8 +1733,18 @@ static int vaapi_device_create(AVHWDeviceContext *ctx, > const char *device, > "/dev/dri/renderD%d", 128 + n); > priv->drm_fd = open(path, O_RDWR); > if (priv->drm_fd < 0) { > - av_log(ctx, AV_LOG_VERBOSE, "Cannot open " > - "DRM render node for device %d.\n", n); > + if (errno == ENOENT) { > + if (n != max_devices - 1) { > + av_log(ctx, AV_LOG_VERBOSE, > + "No render device %s, try next device for > " > + "DRM render node.\n", path); > + continue; > + } else > + av_log(ctx, AV_LOG_VERBOSE, "No avaialbe render > device "
Typo in 'available'. Also, the else is unnecessary since the if() block ends with continue. -- Anton Khirnov _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
