[PATCH umr 1/2] rename mmBIF_BX_PF_MM_* for nbio 2.3

2020-02-14 Thread Xiaojie Yuan
Fixes following error while dumping gfx ring: [BUG]: reg [mmMM_INDEX] not found on asic [navi10] [BUG]: reg [mmMM_INDEX_HI] not found on asic [navi10] [BUG]: reg [mmMM_DATA] not found on asic [navi10] Cannot read from system memory: Operation not permitted [ERROR]: Accessing system memory returned

Re: [PATCH umr 1/2] rename mmBIF_BX_PF_MM_* for nbio 2.3

2020-02-14 Thread Tom St Denis
Hi, Thanks for the patch however since the *.i files are machine generate I'd rather like to avoid patches like this (since they will need to be continually applied). The ideal solution is to either patch src/lib/read_vram.c or to patch the kernel headers.  The kernel headers are also likewi

Re: [PATCH umr 1/2] rename mmBIF_BX_PF_MM_* for nbio 2.3

2020-02-14 Thread Yuan, Xiaojie
Thanks Tom. I'm just publishing this quick fix so that I can continue my debugging with umr, and your solution sounds more reasonable ; ) BR, Xiaojie > On Feb 14, 2020, at 10:55 PM, StDenis, Tom wrote: > > Hi, > > Thanks for the patch however since the *.i files are machine generate I'd > r

Re: [PATCH umr 1/2] rename mmBIF_BX_PF_MM_* for nbio 2.3

2020-02-14 Thread Tom St Denis
Both fixes pushed out to master (on gitlab) Thanks again! Tom On 2020-02-14 10:06 a.m., Yuan, Xiaojie wrote: Thanks Tom. I'm just publishing this quick fix so that I can continue my debugging with umr, and your solution sounds more reasonable ; ) BR, Xiaojie On Feb 14, 2020, at 10:55 PM,