David Biancolin comments

Results 100 comments of


                                            David Biancolin

TracerV-based Targets using BaseVitisConfig Deadlock on Cycle 2

Are you certain it's non-tracerv? If so can you provide the cause?

TracerV-based Targets using BaseVitisConfig Deadlock on Cycle 2

Yes, this is consistent with the initial bug report. @PKUZHOU what version of FireSim are you using?

TracerV-based Targets using BaseVitisConfig Deadlock on Cycle 2

Ok thanks.

TracerV-based Targets using BaseVitisConfig Deadlock on Cycle 2

Ok thanks.

TracerV-based Targets using BaseVitisConfig Deadlock on Cycle 2

Recent changes (went in yesterday) i suspect resolved this indirectly. The root cause if i had to guess was that because there was no PCIM, GG/FIRRTL optimized away all of...

Master doesn't properly measure runtime, reports wrong simulation frequency

It certainly smells like it.

Master doesn't properly measure runtime, reports wrong simulation frequency

This is only a problem on zynq platforms; not F1.

[choice] Adds a means to declare a choice group without use

Also there seems to be some consistency referring to groups, cases and options in the code. Do you want me to go through and update the code and docs to...

[choice] Adds a means to declare a choice group without use

> Why would something simple like having the `Case` insert themselves inside a mutable set inside the `Group` not work? The crux of it is `Case` cannot insert itself into...

Makefile comment line-lengths (style discussion)

I'm fine with this for makefiles. _Hard_ veto on applying sub 80 lines to scala files. Edit: comments would be alright.