On Tue, 2017-06-27 at 17:21 -0400, Lyude Paul wrote:
> NAK, I'd rather just go with the idea you had with removing the
> connector reset in chamelium_init()
Okay then, I'll craft v3 in that directon. This is not a big overhead anyways,
and it does not change anything when running subtests separately (which is what
most test infrastructures using igt seem to be doing).
> On Tue, 2017-06-27 at 13:53 +0300, Paul Kocialkowski wrote:
> > This removes the reset call from the reset_state function and renames
> > it
> > to disconnect_connector. Since a call to reset is already done in
> > chamelium_init, there is no need to do an extra reset in each test.
> >
> > This allows reducing the execution time a bit.
> >
> > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@xxxxxxxxxxxxxxx>
> > ---
> > tests/chamelium.c | 22 +++++++++++-----------
> > 1 file changed, 11 insertions(+), 11 deletions(-)
> >
> > diff --git a/tests/chamelium.c b/tests/chamelium.c
> > index 346018a5..41e0e89b 100644
> > --- a/tests/chamelium.c
> > +++ b/tests/chamelium.c
> > @@ -145,17 +145,17 @@ wait_for_connector(data_t *data, struct
> > chamelium_port *port,
> > }
> >
> > static void
> > -reset_state(data_t *data, struct chamelium_port *port)
> > +disconnect_connector(data_t *data, struct chamelium_port *port)
> > {
> > int p;
> >
> > - chamelium_reset(data->chamelium);
> > -
> > if (port) {
> > + chamelium_unplug(data->chamelium, port);
> > wait_for_connector(data, port,
> > DRM_MODE_DISCONNECTED, false);
> > } else {
> > for (p = 0; p < data->port_count; p++) {
> > port = data->ports[p];
> > + chamelium_unplug(data->chamelium, port);
> > wait_for_connector(data, port,
> > DRM_MODE_DISCONNECTED,
> > false);
> > }
> > @@ -168,7 +168,7 @@ test_basic_hotplug(data_t *data, struct
> > chamelium_port *port, int toggle_count)
> > struct udev_monitor *mon = igt_watch_hotplug();
> > int i;
> >
> > - reset_state(data, NULL);
> > + disconnect_connector(data, NULL);
> > igt_hpd_storm_set_threshold(data->drm_fd, 0);
> >
> > for (i = 0; i < toggle_count; i++) {
> > @@ -202,7 +202,7 @@ test_edid_read(data_t *data, struct
> > chamelium_port *port,
> > data->chamelium, port, false);
> > uint64_t edid_blob_id;
> >
> > - reset_state(data, port);
> > + disconnect_connector(data, port);
> >
> > chamelium_port_set_edid(data->chamelium, port, edid_id);
> > chamelium_plug(data->chamelium, port);
> > @@ -270,7 +270,7 @@ test_suspend_resume_hpd(data_t *data, struct
> > chamelium_port *port,
> > {
> > struct udev_monitor *mon = igt_watch_hotplug();
> >
> > - reset_state(data, port);
> > + disconnect_connector(data, port);
> >
> > /* Make sure we notice new connectors after resuming */
> > try_suspend_resume_hpd(data, port, state, test, mon, false);
> > @@ -294,7 +294,7 @@ test_suspend_resume_hpd_common(data_t *data, enum
> > igt_suspend_state state,
> > igt_debug("Testing port %s\n",
> > chamelium_port_get_name(port));
> > }
> >
> > - reset_state(data, NULL);
> > + disconnect_connector(data, NULL);
> >
> > /* Make sure we notice new connectors after resuming */
> > try_suspend_resume_hpd(data, NULL, state, test, mon, false);
> > @@ -314,7 +314,7 @@ test_suspend_resume_edid_change(data_t *data,
> > struct chamelium_port *port,
> > {
> > struct udev_monitor *mon = igt_watch_hotplug();
> >
> > - reset_state(data, port);
> > + disconnect_connector(data, port);
> >
> > /* First plug in the port */
> > chamelium_port_set_edid(data->chamelium, port, edid_id);
> > @@ -582,7 +582,7 @@ test_hpd_without_ddc(data_t *data, struct
> > chamelium_port *port)
> > {
> > struct udev_monitor *mon = igt_watch_hotplug();
> >
> > - reset_state(data, port);
> > + disconnect_connector(data, port);
> > igt_flush_hotplugs(mon);
> >
> > /* Disable the DDC on the connector and make sure we still
> > get a
> > @@ -604,7 +604,7 @@ test_hpd_storm_detect(data_t *data, struct
> > chamelium_port *port, int width)
> > int count = 0;
> >
> > igt_require_hpd_storm_ctl(data->drm_fd);
> > - reset_state(data, port);
> > + disconnect_connector(data, port);
> >
> > igt_hpd_storm_set_threshold(data->drm_fd, 1);
> > chamelium_fire_hpd_pulses(data->chamelium, port, width, 10);
> > @@ -629,7 +629,7 @@ static void
> > test_hpd_storm_disable(data_t *data, struct chamelium_port *port,
> > int width)
> > {
> > igt_require_hpd_storm_ctl(data->drm_fd);
> > - reset_state(data, port);
> > + disconnect_connector(data, port);
> >
> > igt_hpd_storm_set_threshold(data->drm_fd, 0);
> > chamelium_fire_hpd_pulses(data->chamelium, port,
--
Paul Kocialkowski <paul.kocialkowski@xxxxxxxxxxxxxxx>
Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
