Jump to content
Martin Barnasconi

Issues with method uvm_sequence_base::set_response_queue_error_report_disabled

Recommended Posts

There are two issues with this method definition.

 

1: This method references to an object called response_queue (with the underscore in the name), but such class is not formally defined in the document. Some other methods mention the response queue as a concept (without underscore in the name) but the definition is not mentioned in section 3.1. I propose to remove the underscore and add the definition to section 3.1.

 

2: The setter/getter of this method seems not in line with the name of the method, which has explicitly the word _disabled in the name. It states that setting the value to 0 disables these errors, while setting it to 1 enables them. But I would expect that when the getter get_response_queue_error_report_disabled returns 1, that it reporting is disabled, and returning 0 means reporting is not disabled. The setter should work in the same way ( value =1 means disabled, value = 0 is not disabled)

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×