Jump to content

mastrick

Members
  • Content Count

    48
  • Joined

  • Last visited

  • Days Won

    1

Everything posted by mastrick

  1. mastrick

    RAL Backdoor write

    I believe the UVM intent of is to associate the (whole) signal "IP.REG.regout" with the 16 bits starting at offset 16 of the data in the reg model. The start and offset do not affect the signal, and so you do need to provide the bit range as you show to associate a slice of the signal with a slice of the internal data. I agree the documentation does not make that clear, and I have filed a Mantis to improve that documentation.
  2. Thanks for posting this issue Rolko. The "else ret_val = 1" is a bug in 1800.2-2017-0.9. It is removed in 1800.2-2017-1.0, a release which was approved last week by the committee and should be available for download very soon.
  3. Anil, The UVM committee knows of this limitation but our workaround suggestion would be exactly what you are doing with creating duplicate maps. A fix is on our to-do list. Thanks for posting, Mark
  4. Thank you for posting Chintan. I see the 1800.2-2017 standard says in "8.3.1.4.1 set_inst_override_by_type and set_inst_override_by_name": That says to me that when you gave your original commands to override master_base_seq twice with different override types, the first one should be applied, just as you saw. The set_type_override variations include a "replace" argument that controls whether the new or the older override has priority, so it can work either way.
  5. The problem is that your example code has the constraint block c_addr with "addr>=1000" but addr is an 8-bit field and so could not hold a value of 1000 or greater.
  6. When you set the default_sequence in uvm_config_db, you specify the phase in which the driver will start the sequence. Just with that specification, you can run the sequence in any phase. get_starting_phase() and set_starting_phase() refer to a phase that will be objected to by the automatic phase objection feature of uvm_sequence. The default value for that starting_phase is the phase in which the sequence gets started (what you specified to the uvm_config_db).
  7. Strangely enough, we have just encountered this issue for the first time today. Thanks Robert for posting! In our case, the nested class does not use the field macros but rather calls uvm_report_info directly. We use nested class in this case because this class is used as a replacement class in the test, and to allow us to compile all tests in one executable without conflict, we require such classes to be nested within the test class that uses it. Guess I'll file a mantis to see if the implementation can tolerate such usage.
  8. I believe you want your test sequence to object to main_phase so that you will not have the physical sequence ended before the test sequence ends.
  9. mastrick

    randomize() with inside syntax

    Also, you might want to keep your randomize() call outside the assert(). Otherwise, simulators may not call the randomize at all if you disable assertions (e.g. to temporarily work around a problem). You can assign the return from randomize() to a variable and then assert that variable.
  10. Hi Dave, If you just had "c.a = 10; p.c = c;", you would certainly expect c.a (and p.c.a) to be 10 afterwards. I think the question is why p.randomize(c. does not mean that c.a is a state variable, so it is still 10 after the randomize. Mark
  11. I would expect that is an expected usage model and I have filed Mantis 5529 https://accellera.mantishub.com/view.php?id=5529 againstthe current IEEE standardization effort.
  12. You are correct. That is Mantis 4427, whose fix did not get into the UVM 1.2 release.
  13. If you are using run-time phasing, you simply need to have your environment start the init sequence in some phase before main (perhaps the configure phase), holding the objection until that sequence completes, and then tell test developers that they must run their sequences in main phase (which is the intent of main phase).
  14. I have created a Mantis item to clarify the description of uvm_component_name_check_visitor.
  15. Can you describe in more detail what you mean by "pre_reset_phase entering twice"? What are the signs?
  16. Our uvm_driver derivatives push values into the RTL via clocking blocks in interfaces. This updates the signals synchronously as we typically want, but is there a recommended way to add an asynchronous update of the signal to model a change that could occur when asynchronous reset is applied? We see that if we just assign to the signal at the interface level (not the clocking block level), we get a conflict between both assignments.
  17. Thanks for the suggestion, David. The situation that caused us to consider this question was that we had a verification component modeling an external interface to our ASIC, so it had its own clock, not shared with the rest of the design. There is a possibility that clock is not running, which we wanted to model, but when the clock did not run, our drive through the clocking block never happened. In real circuitry, even though the clock is not running, the asynchronous reset (applied to the external component also) would ensure that the external data was driven to a known value. This made us consider that often our verification components act as if they are synchronously reset but they may not be in real life, and one can imagine scenarios where a bug could escape. So your example implies that a non-blockjng assignment to DATA would change the value on DATA and not be treated as another driver to be resolved on DATA. If so, can you say why the internal assignment behaves differently from a non-blocking assignment from outside the interface of the form m_if_instance.DATA <= value?
  18. The +uvm_set_default_sequence is applied at the time that +uvm_set_config_int is applied, which seems consistent. This is before run_phase, which is why your code overrides the command line value. What do you think about making your code check whether there is an existing setting first? task run_phase(uvm_phase phase); if (!uvm_config_db #(uvm_object_wrapper)::exists(null, "uvm_test_top.my_sequencer.main_phase", "default_sequence")) uvm_config_db #(uvm_object_wrapper)::set(null, "uvm_test_top.my_sequencer.main_phase", "default_sequence", seq_b::type_id::get()); sequencer.start_phase_sequence(phase); endtask: run_phase
  19. It might make a difference which context you saw "sequence driver" in, but the eRM (Specman) concept of "sequence driver" is the same as the UVM sequencer.
  20. mastrick

    UVM Phase issue

    Can you show some of the code you have for the base test pre_configure phase, including the raising of the objection?
  21. It is very easy to assign one of your interfaces into a different domain that is completely independent from the original domain (the only points of synchronization will be the beginning and end of the run_phase). Simply create a new domain and use uvm_component::set_domain() to associate it with the component. uvm_domain new_domain = new ; root_of_interface.set_domain(new_domain); // default is to apply this to all lower levels of hierarchy If you do not want these two domains to be completely independent, but instead act as if they were in the same domain with the exception that one can jump back to pre_reset without the other, then you need to synchronize the domains. domain1.sync(domain2) ; We find the phase jump to be more convenient than the virtual sequence because the virtual sequence has to be implemented by the testbench integrator taking into account each component, while the phase behavior can be specified by the component implementor and used as-is by the testbench integrator. With either approach, the components must respond appropriately to a midsim reset. We recommend the components have a monitor of the reset signal and clear all status appropriately based on that. Any sequences that should be stopped must be taken care of. Sequence behavior is the biggest gotcha of the current phase jumping as it terminates abruptly any sequences started from a phase task or via default_sequence in the config_db. If you already have midsim reset working ok with one domain, you are probably OK there.
  22. There is a Mantis already filed to fix this problem http://www.eda-stds.org/mantis/view.php?id=3901
  23. mastrick

    Typo of uvm_port_base

    You are correct. I will submit an issue to remove the second block.
  24. We weren't going for "totally" different ... I hope the uvm1.1c behavior matches what you want.
  25. I think you want to remove the *. at the front of your "set" path, i.e. uvm_config_db#(uvm_object_wrapper)::set(this,"env.cfg_agt.cfg_sqr.main_phase", "default_sequence", chpp_config_sequence::type_id::get()); as env is already in the context of "this". If that still does not work, turning on the config trace messages will show you what the sequencer is looking for when it queries for default_sequence in the uvm base layer.
×