Hi Matt,
The following condition is missing in t_allocateTBE, but the corepair sends
a message with VicDirty - CoherenceRequestType.
if (in_msg.Type == CoherenceRequestType:VicDirty) {
tbe.DataBlk = in_msg.DataBlk;
}
P.S.: I am not sure whether the complete block should be replaced or just
partially copied.
Thanks,
Sampad
On Sat, Oct 23, 2021 at 2:44 PM Matt Sinclair <[email protected]>
wrote:
> (Resending to mailing list)
>
> Hi Sampad,
>
> There are lines directly below the one I pointed to that do potentially
> overwrite the data there. But I am not 100% sure -- Brad and Matt P, CC'd
> may know better or see something I'm missing.
>
> Matt
>
> On Sat, Oct 23, 2021 at 1:37 PM Sampad Mohapatra <[email protected]> wrote:
>
>> Yes, but the data is coming from the directory and not the incoming
>> message, which has the actual data.
>>
>> Should it not be:
>> *tbe.DataBlk := in_msg.DataBlk;*
>>
>> i.e., store the dirty victim block data in the tbe.
>>
>> Thanks,
>> Sampad
>>
>> On Sat, Oct 23, 2021 at 1:00 PM Matt Sinclair <
>> [email protected]> wrote:
>>
>>> I am not sure I understand completely what you're getting at, but it
>>> appears the allocation of the TBE entry does store the data:
>>> https://gem5.googlesource.com/public/gem5/+/refs/heads/develop/src/mem/ruby/protocol/MOESI_AMD_Base-dir.sm#878
>>>
>>> Matt
>>>
>>> On Thu, Oct 21, 2021 at 11:08 PM Sampad Mohapatra via gem5-users <
>>> [email protected]> wrote:
>>>
>>>> Hello All,
>>>>
>>>> I was looking at the MOESI_AMD_Base-CorePair.sm and
>>>> MOESI_AMD_Base-dir.sm and am not quite sure if the following sequence of
>>>> events are correct or not. Can you please verify?
>>>>
>>>> /////////////////////////
>>>> At CorePair -> invokes action "vd_victim", which sends a data block
>>>> with outgoing message.
>>>>
>>>> At Directory -> undergoes "transition(U, VicDirty, BL)" on message
>>>> reception, but doesn't store the received data block in the generated TBE
>>>> and the message is popped out/discarded.
>>>> /////////////////////////
>>>>
>>>> Is the above expected behaviour ?
>>>>
>>>> Thanks and regards,
>>>> Sampad Mohapatra
>>>> _______________________________________________
>>>> gem5-users mailing list -- [email protected]
>>>> To unsubscribe send an email to [email protected]
>>>> %(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
>>>
>>>
_______________________________________________
gem5-users mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
