Skip to content

Commit

Permalink
[alert,dv] Use priorities in alert agent arbitration
Browse files Browse the repository at this point in the history
I was trying to be clever when I wrote a771270, and set the ping
sequence to run with a lower priority than the alert response
sequence. But this doesn't work with the default fifo arbitration...

Signed-off-by: Rupert Swarbrick <[email protected]>
  • Loading branch information
rswarbrick committed Nov 27, 2024
1 parent 36c283e commit a81ab23
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions hw/dv/sv/alert_esc_agent/alert_esc_agent.sv
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,10 @@ class alert_esc_agent extends dv_base_agent#(
cfg.vif.clk_rst_async_if.set_freq_mhz($urandom_range(10, 240));
end
end

// Configure the sequencer to take priorities into account. This allows alert responses to be
// scheduled over ping requests, which come from a sequence that is started with low priority.
sequencer.set_arbitration(UVM_SEQ_ARB_STRICT_FIFO);
endfunction

// Create automatic response (from monitor) to ping and alert requests.
Expand Down

0 comments on commit a81ab23

Please sign in to comment.