Skip to content

Commit b02612b

Browse files
committed
Add test for instrumentation inquiry
1 parent 861f518 commit b02612b

File tree

3 files changed

+50
-6
lines changed

3 files changed

+50
-6
lines changed

test/1.2/registers/T_instrumentation.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
# © 2021-2023 Intel Corporation
22
# SPDX-License-Identifier: MPL-2.0
33

4+
import instrumentation_access_initiator
45
import instrumentation_access_inquire
56
import instrumentation_access_set_missed
67
import instrumentation_access_set_offset
@@ -23,10 +24,8 @@
2324
import instrumentation_remove_connection_callbacks
2425
import instrumentation_subscribe_multiple
2526

26-
subscribe_b1 = SIM_get_port_interface(
27-
obj, 'bank_instrumentation_subscribe', 'b1')
28-
subscribe_b2 = SIM_get_port_interface(
29-
obj, 'bank_instrumentation_subscribe', 'b2')
27+
subscribe_b1 = obj.bank.b1.iface.bank_instrumentation_subscribe
28+
subscribe_b2 = obj.bank.b2.iface.bank_instrumentation_subscribe
3029
order_b1 = SIM_get_port_interface(obj, 'instrumentation_order', 'b1')
3130

3231
subscribe_ba = [
@@ -37,6 +36,7 @@
3736
# order test first
3837
instrumentation_connection_order.test(obj, subscribe_b1, order_b1)
3938

39+
instrumentation_access_initiator.test(obj, subscribe_b2)
4040
instrumentation_access_inquire.test(obj, subscribe_b2)
4141
instrumentation_access_set_missed.test(obj, subscribe_b1)
4242
instrumentation_access_set_offset.test(obj, subscribe_b1)

test/1.4/registers/instrumentation_test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
# © 2021-2023 Intel Corporation
22
# SPDX-License-Identifier: MPL-2.0
33

4+
import instrumentation_access_initiator
45
import instrumentation_access_inquire
56
import instrumentation_access_set_missed
67
import instrumentation_access_set_offset
@@ -38,8 +39,7 @@ def test(obj):
3839
# order test first
3940
instrumentation_connection_order.test(obj, subscribe_b1, order_b1)
4041

41-
# Disabled tests below broke at some point around 84eff65716f while
42-
# the instrumentation tests were disabled
42+
instrumentation_access_initiator.test(obj, subscribe_b2)
4343
instrumentation_access_inquire.test(obj, subscribe_b2)
4444
with stest.allow_log_mgr(None, 'spec-viol'):
4545
instrumentation_access_set_missed.test(obj, subscribe_b1)
Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,44 @@
1+
# © 2023 Intel Corporation
2+
# SPDX-License-Identifier: MPL-2.0
3+
4+
import conf
5+
import dev_util
6+
import stest
7+
8+
def test(obj, provider):
9+
calls = []
10+
handles = [
11+
provider.register_before_read(
12+
None, 0, 4,
13+
lambda _, access, handle, _2: calls.append(
14+
('br', access.initiator(handle))),
15+
None),
16+
provider.register_after_read(
17+
None, 0, 4,
18+
lambda _, access, handle, _2: calls.append(
19+
('ar', access.initiator(handle))),
20+
None),
21+
provider.register_before_write(
22+
None, 0, 4,
23+
lambda _, access, handle, _2: calls.append(
24+
('bw', access.initiator(handle))),
25+
None),
26+
provider.register_after_write(
27+
None, 0, 4,
28+
lambda _, access, handle, _2: calls.append(
29+
('aw', access.initiator(handle))),
30+
None)]
31+
32+
for (reg, ini) in [
33+
(dev_util.Register_LE(obj.bank.b2, 0, 4), None),
34+
(dev_util.Register_LE(obj.bank.b2, 0, 4, initiator=conf.sim),
35+
conf.sim)]:
36+
stest.expect_equal(reg.read(), 4)
37+
stest.expect_equal(calls, [('br', ini), ('ar', ini)])
38+
del calls[:]
39+
reg.write(4)
40+
stest.expect_equal(calls, [('bw', ini), ('aw', ini)])
41+
del calls[:]
42+
43+
for handle in handles:
44+
provider.remove_callback(handle)

0 commit comments

Comments
 (0)