Re: [RFC][PATCH 3/5] libdrm: Avoid additional drm open close

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Apr 20, 2019 at 12:25 AM John Stultz <john.stultz@xxxxxxxxxx> wrote:
>
> From: Prabhanjan Kandula <pkandula@xxxxxxxxxxxxxx>
>
> Avoid additional drm device open and close.
>
> Cc: Emil Velikov <emil.velikov@xxxxxxxxxxxxx>
> Cc: Sean Paul <seanpaul@xxxxxxxxxxxx>
> Cc: Alistair Strachan <astrachan@xxxxxxxxxx>
> Cc: Marissa Wall <marissaw@xxxxxxxxxx>
> Signed-off-by: John Stultz <john.stultz@xxxxxxxxxx>

Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

> ---
>  xf86drm.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/xf86drm.c b/xf86drm.c
> index fe822ca..2c19376 100644
> --- a/xf86drm.c
> +++ b/xf86drm.c
> @@ -750,8 +750,8 @@ drm_public int drmOpen(const char *name, const char *busid)
>   */
>  drm_public int drmOpenWithType(const char *name, const char *busid, int type)
>  {
> -    if (!drmAvailable() && name != NULL && drm_server_info &&
> -        drm_server_info->load_module) {
> +    if (name != NULL && drm_server_info &&
> +        drm_server_info->load_module && !drmAvailable()) {
>          /* try to load the kernel module */
>          if (!drm_server_info->load_module(name)) {
>              drmMsg("[drm] failed to load kernel module \"%s\"\n", name);
> --
> 2.7.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux