~lttng/lttng-ust/lttng-ust

« back to all changes in this revision

Viewing changes to tests/unit/libcommon/test_smp.c

  • Committer: Mathieu Desnoyers
  • Author(s): Michael Jeanson
  • Date: 2022-07-29 14:32:02 UTC
  • Revision ID: git-v1:a616fb4e0a0245ae2cbd4614910480df19aa23d6
Clarify terminolgy around cpu ids and array length

Rename 'num_possible_cpus' to 'possible_cpus_array_len' to make it
clearer that we use this value to create arrays of per-CPU elements.

Change-Id: Ie5dc9293a95bf321f8add7e9c44ac677bc1fe539
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Show diffs side-by-side

added added

removed removed

Lines of Context:
21
21
};
22
22
 
23
23
static struct parse_test_data parse_test_data[] = {
24
 
        { "", 0 },
25
 
        { "abc", 0 },
26
 
        { ",,,", 0 },
27
 
        { "--", 0 },
28
 
        { ",", 0 },
29
 
        { "-", 0 },
30
 
        { "2147483647", 0 },
31
 
        { "18446744073709551615", 0 },
32
 
        { "0-2147483647", 0 },
33
 
        { "0-18446744073709551615", 0 },
34
 
        { "0", 1 },
35
 
        { "1", 2 },
36
 
        { "0-1", 2 },
37
 
        { "1-3", 4 },
38
 
        { "0,2", 3 },
39
 
        { "1,2", 3 },
40
 
        { "0,4-6,127", 128 },
41
 
        { "0-4095", 4096 },
 
24
        { "", -1 },
 
25
        { "abc", -1 },
 
26
        { ",,,", -1 },
 
27
        { "--", -1 },
 
28
        { ",", -1 },
 
29
        { "-", -1 },
 
30
        { "2147483647", -1 },
 
31
        { "18446744073709551615", -1 },
 
32
        { "0-2147483647", -1 },
 
33
        { "0-18446744073709551615", -1 },
 
34
        { "0", 0 },
 
35
        { "1", 1 },
 
36
        { "0-1", 1 },
 
37
        { "1-3", 3 },
 
38
        { "0,2", 2 },
 
39
        { "1,2", 2 },
 
40
        { "0,4-6,127", 127 },
 
41
        { "0-4095", 4095 },
42
42
 
43
 
        { "\n", 0 },
44
 
        { "abc\n", 0 },
45
 
        { ",,,\n", 0 },
46
 
        { "--\n", 0 },
47
 
        { ",\n", 0 },
48
 
        { "-\n", 0 },
49
 
        { "2147483647\n", 0 },
50
 
        { "18446744073709551615\n", 0 },
51
 
        { "0-2147483647\n", 0 },
52
 
        { "0-18446744073709551615\n", 0 },
53
 
        { "0\n", 1 },
54
 
        { "1\n", 2 },
55
 
        { "0-1\n", 2 },
56
 
        { "1-3\n", 4 },
57
 
        { "0,2\n", 3 },
58
 
        { "1,2\n", 3 },
59
 
        { "0,4-6,127\n", 128 },
60
 
        { "0-4095\n", 4096 },
 
43
        { "\n", -1 },
 
44
        { "abc\n", -1 },
 
45
        { ",,,\n", -1 },
 
46
        { "--\n", -1 },
 
47
        { ",\n", -1 },
 
48
        { "-\n", -1 },
 
49
        { "2147483647\n", -1 },
 
50
        { "18446744073709551615\n", -1 },
 
51
        { "0-2147483647\n", -1 },
 
52
        { "0-18446744073709551615\n", -1 },
 
53
        { "0\n", 0 },
 
54
        { "1\n", 1 },
 
55
        { "0-1\n", 1 },
 
56
        { "1-3\n", 3 },
 
57
        { "0,2\n", 2 },
 
58
        { "1,2\n", 2 },
 
59
        { "0,4-6,127\n", 127 },
 
60
        { "0-4095\n", 4095 },
61
61
};
62
62
 
63
63
static int parse_test_data_len = sizeof(parse_test_data) / sizeof(parse_test_data[0]);
71
71
        diag("Testing smp helpers");
72
72
 
73
73
        for (i = 0; i < parse_test_data_len; i++) {
74
 
                ret = get_num_possible_cpus_from_mask(parse_test_data[i].buf,
 
74
                ret = get_max_cpuid_from_mask(parse_test_data[i].buf,
75
75
                                strlen(parse_test_data[i].buf));
76
76
                ok(ret == parse_test_data[i].expected,
77
 
                        "get_num_possible_cpus_from_mask '%s', expected: '%d', result: '%d'",
 
77
                        "get_max_cpuid_from_mask '%s', expected: '%d', result: '%d'",
78
78
                        parse_test_data[i].buf, parse_test_data[i].expected, ret);
79
79
        }
80
80
 
81
 
        ok(num_possible_cpus() > 0, "num_possible_cpus (%d > 0)", num_possible_cpus());
 
81
        ok(get_possible_cpus_array_len() > 0, "get_possible_cpus_array_len (%d > 0)", get_possible_cpus_array_len());
82
82
 
83
83
        return exit_status();
84
84
}