Re: [PATCH libdrm 1/3] tests/etnaviv_2d_test: explain the errors

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

 



Am Di., 1. Dez. 2020 um 21:38 Uhr schrieb Lubomir Rintel <lkundrak@xxxxx>:
>
> Just so that it's obvious what failed and why.
>
> Signed-off-by: Lubomir Rintel <lkundrak@xxxxx>

Reviewed-by: Christian Gmeiner <christian.gmeiner@xxxxxxxxx>

> ---
>  tests/etnaviv/etnaviv_2d_test.c | 16 ++++++++++++++--
>  1 file changed, 14 insertions(+), 2 deletions(-)
>
> diff --git a/tests/etnaviv/etnaviv_2d_test.c b/tests/etnaviv/etnaviv_2d_test.c
> index 8dd77b66..eb9dfa59 100644
> --- a/tests/etnaviv/etnaviv_2d_test.c
> +++ b/tests/etnaviv/etnaviv_2d_test.c
> @@ -162,9 +162,16 @@ int main(int argc, char *argv[])
>         drmVersionPtr version;
>         int fd, ret = 0;
>
> -       fd = open(argv[1], O_RDWR);
> -       if (fd < 0)
> +       if (argc < 2) {
> +               fprintf(stderr, "Usage: %s /dev/dri/<device>\n", argv[0]);
>                 return 1;
> +       }
> +
> +       fd = open(argv[1], O_RDWR);
> +       if (fd < 0) {
> +               perror(argv[1]);
> +               return 1;
> +       }
>
>         version = drmGetVersion(fd);
>         if (version) {
> @@ -178,6 +185,7 @@ int main(int argc, char *argv[])
>
>         dev = etna_device_new(fd);
>         if (!dev) {
> +               perror("etna_device_new");
>                 ret = 2;
>                 goto out;
>         }
> @@ -185,18 +193,21 @@ int main(int argc, char *argv[])
>         /* TODO: we assume that core 0 is a 2D capable one */
>         gpu = etna_gpu_new(dev, 0);
>         if (!gpu) {
> +               perror("etna_gpu_new");
>                 ret = 3;
>                 goto out_device;
>         }
>
>         pipe = etna_pipe_new(gpu, ETNA_PIPE_2D);
>         if (!pipe) {
> +               perror("etna_pipe_new");
>                 ret = 4;
>                 goto out_gpu;
>         }
>
>         bmp = etna_bo_new(dev, bmp_size, ETNA_BO_UNCACHED);
>         if (!bmp) {
> +               perror("etna_bo_new");
>                 ret = 5;
>                 goto out_pipe;
>         }
> @@ -204,6 +215,7 @@ int main(int argc, char *argv[])
>
>         stream = etna_cmd_stream_new(pipe, 0x300, NULL, NULL);
>         if (!stream) {
> +               perror("etna_cmd_stream_new");
>                 ret = 6;
>                 goto out_bo;
>         }
> --
> 2.28.0
>
> _______________________________________________
> etnaviv mailing list
> etnaviv@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/etnaviv



-- 
greets
--
Christian Gmeiner, MSc

https://christian-gmeiner.info/privacypolicy
_______________________________________________
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