public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>, igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] [PATCH i-g-t v2] tests/perf: add tests for multi context filtering
Date: Tue, 31 Mar 2020 15:20:52 +0300	[thread overview]
Message-ID: <5a6d2f47-63bb-54f2-4ded-cebe1beb6991@intel.com> (raw)
In-Reply-To: <158565656174.18289.8088590257093001034@build.alporthouse.com>

On 31/03/2020 15:09, Chris Wilson wrote:
> Quoting Lionel Landwerlin (2020-03-31 12:53:13)
>> +       /* Missing DRM_I915_PERF_PROP_CTX_HANDLE_ARRAY. */
>> +       {
>> +               uint32_t context_ids[] = {
>> +                       0,
>> +                       0,
>> +               };
>> +               uint64_t properties[] = {
>> +                       /* Note: we have to specify at least one sample property even
>> +                        * though we aren't interested in samples in this case
>> +                        */
>> +                       DRM_I915_PERF_PROP_SAMPLE_OA, true,
>> +
>> +                       /* OA unit configuration */
>> +                       DRM_I915_PERF_PROP_OA_METRICS_SET, test_set->perf_oa_metrics_set,
>> +                       DRM_I915_PERF_PROP_OA_FORMAT, test_set->perf_oa_format,
>> +
>> +                       /* Note: no OA exponent specified in this case */
>> +
>> +                       DRM_I915_PERF_PROP_CTX_HANDLE_ARRAY_LENGTH, ARRAY_SIZE(context_ids),
>> +               };
>> +               struct drm_i915_perf_open_param param = {
>> +                       .flags = I915_PERF_FLAG_FD_CLOEXEC,
>> +                       .num_properties = sizeof(properties) / 16,
>> +                       .properties_ptr = to_user_pointer(properties),
>> +               };
>> +
>> +               do_ioctl_err(drm_fd, DRM_IOCTL_I915_PERF_OPEN, &param, EFAULT);
>> +       }
>> +}
> Also
> (CTX_HANDLE_ARRAY = -1, CTX_HANDLE_ARRAY_LENGTH = 4)
> (CTX_HANDLE_ARRAY = &context_ids[], !IS_ALIGNED(CTX_HANDLE_ARRAY_LENGTH, 4))
> (CTX_HANDLE_ARRAY = &context_ids[], CTX_HANDLE_ARRAY_LENGTH = -4)
>
> I'll take it as a given that passing unmapped pointers doesn't work :)
> -Chris

Ah... I'll check all of that. Thanks for the reminder :)


-Lionel

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

  reply	other threads:[~2020-03-31 12:20 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-31 11:53 [igt-dev] [PATCH i-g-t v2] tests/perf: add tests for multi context filtering Lionel Landwerlin
2020-03-31 12:09 ` Chris Wilson
2020-03-31 12:20   ` Lionel Landwerlin [this message]
2020-04-01  1:53 ` [igt-dev] ✗ Fi.CI.BUILD: failure for " Patchwork

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=5a6d2f47-63bb-54f2-4ded-cebe1beb6991@intel.com \
    --to=lionel.g.landwerlin@intel.com \
    --cc=chris@chris-wilson.co.uk \
    --cc=igt-dev@lists.freedesktop.org \
    /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