qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v14 02/26] target/loongarch: Add core definition


From: gaosong
Subject: Re: [PATCH v14 02/26] target/loongarch: Add core definition
Date: Wed, 12 Jan 2022 18:17:47 +0800
User-agent: Mozilla/5.0 (X11; Linux loongarch64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0

Hi,

On 2022/1/12 下午5:28, gaosong wrote:
+    data = "" CPUCFG16, L3_IUUNIFY, 1);
+    data = "" CPUCFG16, L3_IUINCL, 1);
+    env->cpucfg[16] = data;
+
+    data = ""
+    data = "" CPUCFG17, L1IU_WAYS, 0x8003);

This seems out-of-place, according to the manual this field is Way-1 for the L1I cache, so you have 0x8004=32772 ways in this cache?

Same for all similar constructions below.

I have time to reply to your comment now.
As in the previous comments, I don't remember which one,these values should be the same as the values of the physical environment. I dumped 'CPUCFG17' value again,
the value is no problem. Maybe you didn't think about dumping these values when you commented. The value of  'L11U_SIZE' is dumped to be 0. and cpucfg[i] has been 
initialized to 0 before. There is no need to set it again.
Not quite right,  cpucfg[17] is '0x60800003', I missed a '0',  I don't know from which version it's wrong
Thank you very much,  if I hadn't dumped the value today, I wouldn't have realized the wrong.  
Thanks
Song

reply via email to

[Prev in Thread] Current Thread [Next in Thread]