Re: [LTP] [PATCH] lib: do_setup(): Check for supported arch first

[Cyril Hrubis <chrubis@suse.cz>]

Hi!
> I guess the reason is that we want to skip code which is not compiling 
> for our architecture, right?
> But do we already have TST_TEST_CONF, so I'm a bit puzzled on the use case.

The problem here is that with TST_TEST_TCONF() the supported architecture
is not exported into the metadata.

There are two cases where TST_TEST_TCONF() is used in LTP:

- usupported architecture for the test
- missing headers during compilation

The first case should ideally be converted so that we have the supported
architectures in the test metadata.

Another option is just to add metadata to the tst_test structure and
keep TST_TEST_TCONF(), but I do not like that solution, because the
ifdefs and .supported_archs can go out of sync easily because tst_test
structure is not used when TST_TEST_TCONF() is used.

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp