<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;" dir="ltr">
<p>HI Emil,</p>
<p><br>
</p>
<p>See below.</p>
<p><br>
</p>
<p>Thanks,</p>
<p>Alex<br>
</p>
<br>
<br>
<div style="color: rgb(0, 0, 0);">
<div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><b>From:</b> Emil Velikov <emil.l.velikov@gmail.com><br>
<b>Sent:</b> Friday, January 20, 2017 8:24 AM<br>
<b>To:</b> Xie, AlexBin<br>
<b>Cc:</b> amd-gfx mailing list<br>
<b>Subject:</b> Re: [PATCH libdrm 2/3] amdgpu: A new option to choose which device to run most tests</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">On 19 January 2017 at 22:53, Alex Xie <AlexBin.Xie@amd.com> wrote:<br>
> This can be used to test multiple GPUs<br>
><br>
> Signed-off-by: Alex Xie <AlexBin.Xie@amd.com><br>
> ---<br>
>  tests/amdgpu/amdgpu_test.c | 25 ++++++++++++++++++++-----<br>
>  1 file changed, 20 insertions(+), 5 deletions(-)<br>
><br>
> diff --git a/tests/amdgpu/amdgpu_test.c b/tests/amdgpu/amdgpu_test.c<br>
> index e42ef9d..2437db4 100644<br>
> --- a/tests/amdgpu/amdgpu_test.c<br>
> +++ b/tests/amdgpu/amdgpu_test.c<br>
> @@ -108,12 +108,14 @@ static void display_test_suites(void)<br>
><br>
><br>
>  /** Help string for command line parameters */<br>
> -static const char usage[] = "Usage: %s [-hl] [<-s <suite id>> [-t <test id>]]\n"<br>
> -                               "where:\n"<br>
> -                               "       l - Display all suites and their tests\n"<br>
> -                               "       h - Display this help\n";<br>
> +static const char usage[] =<br>
> +       "Usage: %s [-hl] [<-s <suite id>> [-t <test id>]] [-d <device_id>]\n"<br>
> +       "where:\n"<br>
> +       "       l - Display all suites and their tests\n"<br>
> +       "       d - Choose which device to run tests\n"<br>
You want to elaborate on what you mean with "device" here.<br>
Even if currently "card0 is my Kabini, card1 Hawaii, card2 other" that<br>
may change upon reboot.<br>
So although very nice to say "0,1, 2..." things are non-deterministic.<br>
<br>
[Alex Bin Xie] Originally I just want to give a simple solution for people to swap to test another GPU device.<br>
Every time amdgpu_test runs, it lists the device id and its information. I was thinking to improve this but such as adding an option to list the devices. So that the 0, 1 ,2 does not change if computer is not reboot. If 0, 1, 2 is not deterministic, I have
 to give up this human friendly interface...<br>
<br>
How about choose the test device by bus ID?  I will add an option to list the devices with BUS ID (if people run drmdevice program, they know the information too). Then people can choose  device on which BUS ID to run the test. I will provide a new patch if
 there is no object to this.<br>
<br>
<br>
-Emil<br>
</div>
</span></font></div>
</div>
</body>
</html>