From: "Hagaragi, Arundhati" <arundhati.hagaragi@intel.com>
To: "Latvala, Petri" <petri.latvala@intel.com>
Cc: "igt-dev@lists.freedesktop.org" <igt-dev@lists.freedesktop.org>
Subject: Re: [igt-dev] [PATCH i-g-t v2] tests/kms_lease: Add subtest description
Date: Mon, 17 May 2021 09:39:35 +0000 [thread overview]
Message-ID: <DM6PR11MB3162CFF10EE790CB8E0B4E798B2D9@DM6PR11MB3162.namprd11.prod.outlook.com> (raw)
In-Reply-To: <YKI00StJlHbQCsPu@platvala-desk.ger.corp.intel.com>
-----Original Message-----
From: Latvala, Petri <petri.latvala@intel.com>
Sent: Monday, May 17, 2021 2:48 PM
To: Hagaragi, Arundhati <arundhati.hagaragi@intel.com>
Cc: igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] [PATCH i-g-t v2] tests/kms_lease: Add subtest description
On Thu, May 13, 2021 at 01:48:32PM +0530, Arundhati Hagaragi wrote:
> Added description for subtest
>
> Signed-off-by: Arundhati Hagaragi <arundhati.hagaragi@intel.com>
> Cc: B S Karthik <karthik.b.s@intel.com>
>
> v2: Modified & applied description to each tests seperately. (Petri)
> ---
> tests/kms_lease.c | 35 +++++++++++++++++++++--------------
> 1 file changed, 21 insertions(+), 14 deletions(-)
>
> diff --git a/tests/kms_lease.c b/tests/kms_lease.c index
> 6397ef8f..0c019a46 100644
> --- a/tests/kms_lease.c
> +++ b/tests/kms_lease.c
> @@ -1281,21 +1281,27 @@ igt_main
> const struct {
> const char *name;
> void (*func)(data_t *);
> + const char *desc;
> } funcs[] = {
> - { "simple_lease", simple_lease },
> - { "lessee_list", lessee_list },
> - { "lease_get", lease_get },
> - { "lease_unleased_connector", lease_unleased_connector },
> - { "lease_unleased_crtc", lease_unleased_crtc },
> - { "lease_revoke", lease_revoke },
> - { "lease_again", lease_again },
> - { "lease_invalid_connector", lease_invalid_connector },
> - { "lease_invalid_crtc", lease_invalid_crtc },
> - { "lease_invalid_plane", lease_invalid_plane },
> - { "page_flip_implicit_plane", page_flip_implicit_plane },
> - { "setcrtc_implicit_plane", setcrtc_implicit_plane },
> - { "cursor_implicit_plane", cursor_implicit_plane },
> - { "atomic_implicit_crtc", atomic_implicit_crtc },
> + { "simple_lease", simple_lease, "Check if create lease ioctl call works correctly" },
> + { "lessee_list", lessee_list, "Check if listed lease is same as corrected one" },
> + { "lease_get", lease_get, "Tests getting the required contents of a lease" },
> + { "lease_unleased_connector", lease_unleased_connector, "Check if func fails on an"
> + "unleased connector" },
> + { "lease_unleased_crtc", lease_unleased_crtc, "Check if func fails on unleased crtc" },
> + { "lease_revoke", lease_revoke, "Tests revocation of lease" },
> + { "lease_again", lease_again, "Tests leasing objects more than once" },
> + { "lease_invalid_connector", lease_invalid_connector, "Tests leasing an invalid connector" },
> + { "lease_invalid_crtc", lease_invalid_crtc, "Tests leasing an invalid crtc" },
> + { "lease_invalid_plane", lease_invalid_plane, "Tests leasing an invalid plane" },
> + { "page_flip_implicit_plane", page_flip_implicit_plane, "Check that page-flip ioclt"
> + "fails correctly on implicit primary plane" },
> + { "setcrtc_implicit_plane", setcrtc_implicit_plane, "Check that setcrtc ioctl fails"
> + "correctly on implicit primary plane" },
> + { "cursor_implicit_plane", cursor_implicit_plane, "Check that setcursor ioctl fails"
> + "correctly on implicit primary plane" },
> + { "atomic_implicit_crtc", atomic_implicit_crtc, "Check that atomic ioctl fails correctly"
> + "on crtc_id property" },
When splitting strings to multiple lines, make sure you still add the space between words. For example cursor_implicit_plane will have "failscorrectly" with this.
--
Petri Latvala
Sure , Will take care of this.
Thanks,
Arundhati
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
next prev parent reply other threads:[~2021-05-17 9:39 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-07 12:50 [igt-dev] [PATCH i-g-t] tests/kms_lease: Add subtest description Arundhati Hagaragi
2021-05-07 13:39 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2021-05-07 14:04 ` [igt-dev] [PATCH i-g-t] " Petri Latvala
2021-05-07 16:53 ` Hagaragi, Arundhati
2021-05-07 16:38 ` [igt-dev] ✓ Fi.CI.IGT: success for " Patchwork
2021-05-13 8:18 ` [igt-dev] [PATCH i-g-t v2] " Arundhati Hagaragi
2021-05-17 9:18 ` Petri Latvala
2021-05-17 9:39 ` Hagaragi, Arundhati [this message]
2021-05-13 9:22 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_lease: Add subtest description (rev2) Patchwork
2021-05-13 11:09 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2021-05-17 10:59 ` [igt-dev] [PATCH i-g-t v3] tests/kms_lease: Add subtest description Arundhati Hagaragi
2021-05-17 11:52 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_lease: Add subtest description (rev3) Patchwork
2021-05-17 17:45 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2021-05-18 8:58 ` [igt-dev] [PATCH i-g-t v4] tests/kms_lease: Add subtest description Arundhati Hagaragi
2021-05-18 9:27 ` Karthik B S
2021-05-18 11:00 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_lease: Add subtest description (rev4) Patchwork
2021-05-18 11:11 ` [igt-dev] [PATCH i-g-t v5] tests/kms_lease: Add subtest description Arundhati Hagaragi
2021-05-18 12:00 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_lease: Add subtest description (rev5) Patchwork
2021-05-18 12:05 ` [igt-dev] [PATCH i-g-t v6] tests/kms_lease: Add subtest description Arundhati Hagaragi
2021-05-18 13:00 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_lease: Add subtest description (rev6) Patchwork
2021-05-18 15:20 ` [igt-dev] ✗ Fi.CI.IGT: failure for tests/kms_lease: Add subtest description (rev4) Patchwork
2021-05-18 20:00 ` [igt-dev] ✗ Fi.CI.IGT: failure for tests/kms_lease: Add subtest description (rev5) Patchwork
2021-05-18 21:45 ` [igt-dev] ✗ Fi.CI.IGT: failure for tests/kms_lease: Add subtest description (rev6) 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=DM6PR11MB3162CFF10EE790CB8E0B4E798B2D9@DM6PR11MB3162.namprd11.prod.outlook.com \
--to=arundhati.hagaragi@intel.com \
--cc=igt-dev@lists.freedesktop.org \
--cc=petri.latvala@intel.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.