search

bespoke-silicon-group / bsg_manycore

Tile based architecture designed for computing efficiency, scalability and generality
Other
221 stars 58 forks source link

bsg_cache word tracking wormhole change #662

Closed tommydcjung closed 1 year ago

tommydcjung commented 2 years ago
mrutt92 commented 1 year ago

what's up with this pr? looks like it's necessary to keep up to date with master in basejump.

taylor-bsg commented 1 year ago

Hi,

I think it would be helpful to have the parameter that allows us to enable/disable it, so that bsg_cache can still be used even when word writes are not available in the memory backing store.

On Mon, Oct 24, 2022 at 4:03 PM Dan Petrisko @.***> wrote:

@.**** approved this pull request.

LGTM!

  • Is it worth it to add a parameter for word tracking, or do we feel the hardware overhead is always justified?
  • Have we tested with this same changeset except for word_tracking_p(0), from a functionality perspective?

— Reply to this email directly, view it on GitHub https://github.com/bespoke-silicon-group/bsg_manycore/pull/662#pullrequestreview-1153918038, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEFG5AGUKAEPVD2MBPDLGYLWE4IU3ANCNFSM546ZGNOA . You are receiving this because you are subscribed to this thread.Message ID: @.*** com>

tommydcjung commented 1 year ago