Re: [sv-ec] Updated Event Scheduling Proposal (section 9.3) with new region descriptions

Hi, Neil -

Comments below.

At 07:06 PM 12/16/2006, Neil Korpusik wrote:
>Hi Cliff,
>
>I have a few comments on your proposed set of changes.
>
>1. 9.3
>
>    There shouldn't be two consecutive PLI regions. Both the Post-NBA and the
>    Pre-postponed regions are PLI regions. Only one of them should be listed
>    in this re-worded paragraph, otherwise you end up with two consecutive
>    PLI regions.

I am not opposed but I am not sure this is true. The proposed 
Post-re-NBA region allows PLI commands to be made after the Re-NBA 
assignments that can be considered if the little loop does indeed 
loop, while the Pre-Postponed PLI region gives a region where PLI 
commands can be executed after all little-loop activity that could 
either feedback into the Active region of the big loop or executed 
commands just before the final Postponed activity of the current time slot.

I don't know if this is important or not (??) If it might be 
important, we can always include the Post-re-NBA PLI region and not 
define any PLI commands to execute in this region and eventually kill 
it if we find it is unnecessary (??)

>    From:
>       "The Pre-active, Active, Inactive, Pre-NBA, NBA, Post-NBA, 
> Pre-postponed
>       and Postponed regions essentially encompass the IEEE 1364 reference
>       model for simulation, with exactly the same level of determinism."
>    To:
>       "The Pre-active, Active, Inactive, Pre-NBA, NBA, Post-NBA
>       and Postponed regions essentially encompass the IEEE 1364 reference
>       model for simulation, with exactly the same level of determinism."

One could argue that the Pre-active, Pre-NBA, Post-NBA and 
Pre-postponed regions did not previously exist, but I believe we 
agreed that PLI commands did execute at those times, which is why 
this paragraph included the words "essentially encompass." This is 
why Arturo and I thought that Pre-postponed belonged in the list.

>2. Figure 9.1
>
>    - It would be better if the new regions were in blue.

Agreed.

>    - I noticed that your version of the figure shows the first letter of
>      each region capitalized. The version of this figure in the LRM shows
>      the first letter in lowercase. If the editor "blindly" adds the
>      new regions as shown in your version we will end up with a 
> capitalization
>      inconsistency.

I thought the old diagram was not consistent with the text, but I 
could be wrong. I will check this harder.

>    - Also note Pre-Postponed versus pre-postponed (two differences in
>      capitalization)

Same as above.

>3. Algorithm in 9.3.1
>
>    There are capitalization inconsistencies here as well.

We can check this.

>4. I agree with the friendly ammendment sent from Dave.
>    It is copied below for your convenience.

I agree.

Regards - Cliff

>Neil
>
>
>
>Dave's feedback:  12/05/06 23:31 - nzk agrees with this suggestion
>
>    Cliff
>
>    I think there is a problem with this sentence on page 2 paragraph 4:
>
>    "A nonblocking assignment specified in a program block schedules the
>    process for resumption in the Re-NBA region."
>
>    I believe it should be
>
>    "A nonblocking assignment specified in a program block creates a re-NBA
>    update event, scheduled for the current or later simulation time."
>
>    Dave
>
>
>
>
>Clifford E. Cummings wrote On 12/05/06 17:46,:
> > Hi, All -
> >
> > Updated proposal for event scheduling of new event regions. Arturo
> > also reviewed and blessed the proposal.
> >
> > Regards - Cliff
> >
> > ----------------------------------------------------
> > Cliff Cummings - Sunburst Design, Inc.
> > 14314 SW Allen Blvd., PMB 501, Beaverton, OR 97005
> > Phone: 503-641-8446 / FAX: 503-641-8486
> > cliffc@sunburst-design.com / www.sunburst-design.com
> > Expert Verilog, SystemVerilog, Synthesis and Verification Training
>
>--
>---------------------------------------------------------------------
>Neil Korpusik                                     Tel: 408-720-4852
>Senior Staff Engineer                             Fax: 408-720-4850
>Frontend Technologies - ASICs & Processors (FTAP)
>Sun Microsystems
>email: neil.korpusik@sun.com
>---------------------------------------------------------------------

----------------------------------------------------
Cliff Cummings - Sunburst Design, Inc.
14314 SW Allen Blvd., PMB 501, Beaverton, OR 97005
Phone: 503-641-8446 / FAX: 503-641-8486
cliffc@sunburst-design.com / www.sunburst-design.com
Expert Verilog, SystemVerilog, Synthesis and Verification Training