public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
From: zhoucm1 <zhoucm1@amd.com>
To: "Daniel Vetter" <daniel.vetter@ffwll.ch>,
	"Chunming Zhou" <David1.Zhou@amd.com>,
	"Christian König" <ckoenig.leichtzumerken@gmail.com>,
	"Alex Deucher" <alexander.deucher@amd.com>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>,
	amd-gfx list <amd-gfx@lists.freedesktop.org>,
	IGT development <igt-dev@lists.freedesktop.org>,
	dri-devel <dri-devel@lists.freedesktop.org>,
	Sean Paul <sean@poorly.run>
Subject: Re: [igt-dev] [PATCH] RFC: Make igts for cross-driver stuff mandatory?
Date: Fri, 26 Oct 2018 16:48:57 +0800	[thread overview]
Message-ID: <a693fa3f-3c7f-c557-68a1-c2b1644fdf11@amd.com> (raw)
In-Reply-To: <CAKMK7uFsET+riNKw6eGt1+Uiuhq8eww1E4OFYVCC83CfQFN5qg@mail.gmail.com>



On 2018年10月26日 16:32, Daniel Vetter wrote:
> On Fri, Oct 26, 2018 at 5:50 AM Zhou, David(ChunMing)
> <David1.Zhou@amd.com> wrote:
>> Make igt for cross-driver, I think you should rename it first, not an intel specific. NO company wants their employee working on other company stuff.
>> You can rename it to DGT(drm graphics test), and published following  libdrm, or directly merge to libdrm, then everyone  can use it and develop it in same page, which is only my personal opinion.
> We renamed it ot  IGT gpu tools, that was even enough for ARM folks.
> If this is seriously what AMD expects before considering, I'm not sure
> what to say.
>
> Alex, Christian, is this the official AMD stance that you can't touch
> stuff because of the letter i?
Nope, as I said last, this is just my personal thought. And I'm not sure 
what opinion of others.

-David
> -Daniel
>
>
>> Regards,
>> David
>>
>>> -----Original Message-----
>>> From: dri-devel <dri-devel-bounces@lists.freedesktop.org> On Behalf Of Eric
>>> Anholt
>>> Sent: Friday, October 26, 2018 12:36 AM
>>> To: Sean Paul <sean@poorly.run>; Daniel Vetter <daniel.vetter@ffwll.ch>
>>> Cc: IGT development <igt-dev@lists.freedesktop.org>; Intel Graphics
>>> Development <intel-gfx@lists.freedesktop.org>; DRI Development <dri-
>>> devel@lists.freedesktop.org>; amd-gfx@lists.freedesktop.org
>>> Subject: Re: [igt-dev] [PATCH] RFC: Make igts for cross-driver stuff
>>> mandatory?
>>>
>>> Sean Paul <sean@poorly.run> writes:
>>>
>>>> On Fri, Oct 19, 2018 at 10:50:49AM +0200, Daniel Vetter wrote:
>>>>> Hi all,
>>>>>
>>>>> This is just to collect feedback on this idea, and see whether the
>>>>> overall dri-devel community stands on all this. I think the past few
>>>>> cross-vendor uapi extensions all came with igts attached, and
>>>>> personally I think there's lots of value in having them: A
>>>>> cross-vendor interface isn't useful if every driver implements it
>>>>> slightly differently.
>>>>>
>>>>> I think there's 2 questions here:
>>>>>
>>>>> - Do we want to make such testcases mandatory?
>>>>>
>>>> Yes, more testing == better code.
>>>>
>>>>
>>>>> - If yes, are we there yet, or is there something crucially missing
>>>>>    still?
>>>> In my experience, no. Last week while trying to replicate an intel-gfx
>>>> CI failure, I tried compiling igt for one of my (intel) chromebooks.
>>>> It seems like cross-compilation (or, in my case, just specifying
>>>> prefix/ld_library_path/sbin_path) is broken on igt. If we want to
>>>> impose restrictions across the entire subsystem, we need to make sure
>>>> that everyone can build and deploy igt easily.
>>>>
>>>> I managed to hack around everything and get it working, but I still
>>>> haven't tried switching out the toolchain. Once we have some GitLab CI
>>>> to validate cross-compilation, then we can consider making IGT mandatory.
>>>>
>>>> It's possible that I'm just a meson n00b and didn't use the right
>>>> incantation, so maybe it already works, but then we need better
>>> documentation.
>>>> I've pasted my horrible hacks below, I also didn't have libunwind, so
>>>> removed its usage.
>>> I've also had to cut out libunwind for cross-compiling on many occasions.
>>> Worst library.
>
>

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2018-10-26  8:48 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-19  8:50 [igt-dev] [PATCH] RFC: Make igts for cross-driver stuff mandatory? Daniel Vetter
2018-10-25  9:58 ` Liviu Dudau
2018-10-25 10:31   ` [Intel-gfx] " Daniel Vetter
2018-10-25 12:51 ` [igt-dev] " Sean Paul
2018-10-25 16:35   ` [Intel-gfx] " Eric Anholt
2018-10-26  3:49     ` Zhou, David(ChunMing)
2018-10-26  8:32       ` Daniel Vetter
2018-10-26  8:48         ` zhoucm1 [this message]
2018-10-26 14:41         ` Alex Deucher
2018-10-26 15:28 ` Alex Deucher
2018-10-30  2:17 ` [igt-dev] [Intel-gfx] " Dave Airlie
2018-10-30  8:54   ` Daniel Vetter

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=a693fa3f-3c7f-c557-68a1-c2b1644fdf11@amd.com \
    --to=zhoucm1@amd.com \
    --cc=David1.Zhou@amd.com \
    --cc=alexander.deucher@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=ckoenig.leichtzumerken@gmail.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=igt-dev@lists.freedesktop.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=sean@poorly.run \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox