Hi Sampad,

The AMD folks that are CC'd are better people to comment than me, but I
believe the L3 cache is a "memory side" cache, and thus doesn't need to
maintain coherence.

If you have a fix to add hits/misses for any of these files, and are
willing to contribute it back, it would be great if you could submit a
patch for it.

Matt

On Mon, Aug 17, 2020 at 11:00 PM Sampad Mohapatra <su...@psu.edu> wrote:

> Hi Matt,
>
> I am currently trying to add the stats by myself. I also noticed that
> neither hit nor miss stats are updated for L3 Cache.
>
> But, I am facing a different issue now. The directory ctrl (DirCntrl) in
> GPU_VIPER.py has a L3 cache (L3Cache).
> But, there is also a separate L3Cntrl class which inherits from
> L3Cache_Controller, but it is unused.
> The L3Cache_Controller is generated from MOESI_AMD_Base-L3cache.sm, which
> is a part of the Viper protocol.
>
> If the L3Cache_Controller isn't used, then why is it a part of the Viper
> protocol ?
> Does the L3 Cache not maintain any coherency ?
> Is this the intended behaviour of the default configuration ?
>
> Thanks and Regards,
> Sampad
>
> On Thu, Aug 13, 2020 at 5:40 PM Matt Sinclair via gem5-users <
> gem5-users@gem5.org> wrote:
>
>> Hi Sampad,
>>
>> I'm not aware of a patch for this.  There was recently a patch to add
>> similar support for the VIPER protocol:
>> https://gem5-review.googlesource.com/c/public/gem5/+/30174.  If the AMD
>> folks (CC'd) don't have a patch, then the next best thing would be to do
>> something similar to the VIPER patch (although yes, the Core Pair changes
>> would be a little more complicated).
>>
>> Thanks,
>> Matt
>>
>> On Tue, Aug 4, 2020 at 4:53 PM Sampad Mohapatra via gem5-users <
>> gem5-users@gem5.org> wrote:
>>
>>> Hello All,
>>>
>>> MOESI AMD Base - CorePair state machine is missing the actions for L1
>>> and L2 hit statistics.
>>> The stats are present, but since no "action" is created nor used
>>> (actions to update misses are present for both L1 and L2), the stats stay
>>> at 0.
>>>
>>> I am not clear as to which state transitions should
>>> update the hit stats. Is there a patch for this ?
>>>
>>> Thank You,
>>> Sampad Mohapatra
>>>
>>>
>>> <http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail>
>>>  Virus-free.
>>> www.avg.com
>>> <http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail>
>>> <#m_-8358212102347321869_m_-6648754351699623829_m_1723920294658796898_m_-8550086026140872479_m_-203124547492808951_m_-2486225155787835700_m_2217726199504402898_DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
>>> _______________________________________________
>>> gem5-users mailing list -- gem5-users@gem5.org
>>> To unsubscribe send an email to gem5-users-le...@gem5.org
>>> %(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
>>
>> _______________________________________________
>> gem5-users mailing list -- gem5-users@gem5.org
>> To unsubscribe send an email to gem5-users-le...@gem5.org
>> %(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
>
>
_______________________________________________
gem5-users mailing list -- gem5-users@gem5.org
To unsubscribe send an email to gem5-users-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to