mirror of
https://github.com/tiagovignatti/intel-gpu-tools.git
synced 2025-06-10 09:26:10 +00:00
lib: make igt_set_vt_graphics_mode never fail
All tests agree that this is a letal failure, so no point to pass it back to callers. Also add some igt_require calls for the drm_fd where I've noticed that it's missing. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
This commit is contained in:
parent
0369fe19b8
commit
1f0cf2df85
@ -2111,10 +2111,9 @@ static void restore_vt_mode_at_exit(int sig)
|
||||
* original mode.
|
||||
*/
|
||||
|
||||
int igt_set_vt_graphics_mode(void)
|
||||
void igt_set_vt_graphics_mode(void)
|
||||
{
|
||||
if (igt_install_exit_handler(restore_vt_mode_at_exit))
|
||||
return -1;
|
||||
do_or_die(igt_install_exit_handler(restore_vt_mode_at_exit));
|
||||
|
||||
igt_disable_exit_handler();
|
||||
orig_vt_mode = set_vt_mode(KD_GRAPHICS);
|
||||
@ -2122,7 +2121,7 @@ int igt_set_vt_graphics_mode(void)
|
||||
orig_vt_mode = -1UL;
|
||||
igt_enable_exit_handler();
|
||||
|
||||
return orig_vt_mode < 0 ? -1 : 0;
|
||||
igt_assert(orig_vt_mode >= 0);
|
||||
}
|
||||
|
||||
int kmstest_get_connector_default_mode(int drm_fd, drmModeConnector *connector,
|
||||
|
@ -394,7 +394,7 @@ void igt_enable_exit_handler(void);
|
||||
void igt_disable_exit_handler(void);
|
||||
|
||||
/* set vt into graphics mode, required to prevent fbcon from interfering */
|
||||
int igt_set_vt_graphics_mode(void);
|
||||
void igt_set_vt_graphics_mode(void);
|
||||
|
||||
/* prefault disabling, needs the corresponding debugfs interface */
|
||||
void igt_disable_prefault(void);
|
||||
|
@ -220,7 +220,9 @@ int main(int argc, char **argv)
|
||||
const char *cmd = "pipe A none";
|
||||
|
||||
data.drm_fd = drm_open_any();
|
||||
do_or_die(igt_set_vt_graphics_mode());
|
||||
igt_require(data.drm_fd >= 0);
|
||||
|
||||
igt_set_vt_graphics_mode();
|
||||
|
||||
display_init(&data);
|
||||
|
||||
|
@ -334,7 +334,9 @@ int main(int argc, char **argv)
|
||||
const char *cmd = "pipe A none";
|
||||
|
||||
data.drm_fd = drm_open_any();
|
||||
do_or_die(igt_set_vt_graphics_mode());
|
||||
igt_require(data.drm_fd >= 0);
|
||||
|
||||
igt_set_vt_graphics_mode();
|
||||
|
||||
igt_debugfs_init(&data.debugfs);
|
||||
data.ctl = igt_debugfs_fopen(&data.debugfs,
|
||||
|
@ -1493,7 +1493,7 @@ int main(int argc, char **argv)
|
||||
igt_fixture {
|
||||
drm_fd = drm_open_any();
|
||||
|
||||
do_or_die(igt_set_vt_graphics_mode());
|
||||
igt_set_vt_graphics_mode();
|
||||
do_or_die(igt_install_exit_handler(kms_flip_exit_handler));
|
||||
get_timestamp_format();
|
||||
|
||||
|
@ -242,7 +242,7 @@ int main(int argc, char **argv)
|
||||
devid = intel_get_drm_devid(drm_fd);
|
||||
batch = intel_batchbuffer_alloc(bufmgr, devid);
|
||||
|
||||
do_or_die(igt_set_vt_graphics_mode());
|
||||
igt_set_vt_graphics_mode();
|
||||
}
|
||||
|
||||
for (i = 0; i < ARRAY_SIZE(tests); i++) {
|
||||
|
@ -702,7 +702,7 @@ int main(int argc, char **argv)
|
||||
igt_fixture {
|
||||
drm_fd = drm_open_any();
|
||||
if (!dry_run)
|
||||
do_or_die(igt_set_vt_graphics_mode());
|
||||
igt_set_vt_graphics_mode();
|
||||
|
||||
drm_resources = drmModeGetResources(drm_fd);
|
||||
assert(drm_resources);
|
||||
|
@ -292,7 +292,7 @@ static void init_mode_set_data(struct mode_set_data *data)
|
||||
data->bufmgr = drm_intel_bufmgr_gem_init(drm_fd, 4096);
|
||||
data->devid = intel_get_drm_devid(drm_fd);
|
||||
|
||||
do_or_die(igt_set_vt_graphics_mode());
|
||||
igt_set_vt_graphics_mode();
|
||||
drm_intel_bufmgr_gem_enable_reuse(data->bufmgr);
|
||||
}
|
||||
|
||||
|
@ -801,7 +801,7 @@ int main(int argc, char **argv)
|
||||
goto out_close;
|
||||
}
|
||||
|
||||
do_or_die(igt_set_vt_graphics_mode());
|
||||
igt_set_vt_graphics_mode();
|
||||
|
||||
mainloop = g_main_loop_new(NULL, FALSE);
|
||||
if (!mainloop) {
|
||||
|
Loading…
x
Reference in New Issue
Block a user