Skip to content

virtual address is different with physical address in some CPU - #2

Closed
qepdclqu wants to merge 1 commit into
eclipse-threadx:masterfrom
qepdclqu:master
Closed

virtual address is different with physical address in some CPU#2
qepdclqu wants to merge 1 commit into
eclipse-threadx:masterfrom
qepdclqu:master

Conversation

@qepdclqu

Copy link
Copy Markdown

您好,最近我在移植usbx到一个芯片上时,发现usbx上可能存在"bug"。usbx上有两个接口_ux_utility_virtual_address和_ux_utility_physical_address,里面的虚拟地址和物理地址是一一映射的,代码中有很多地方也用到了这两个接口用来实现CPU和HC各自访问所需要的地址空间,但是我发现有些源文件的代码中并没有遵守这样的规则,因为我的芯片上的CPU使用的虚拟地址和物理地址并不是一一映射的,当CPU使用物理地址时,会有异常,而HC使用虚拟地址时也会存在问题(我相信在某些虚拟地址和物理地址不是一一映射的CPU上也会存在同样的问题)。
我的描述是否能说明白我遇到的问题,不知道您那边是否认为这些是"bug"。
所以,我找出了四个文件并进行了修改,如果是“bug”,我相信usbx中并不是只有这4个文件存在这样的问题。
我第一次使用github,所以我将更改的直接提交到了master分支,希望不会造成一些困扰。
以上,谢谢。
p.s. 我希望学习和使用usbx,对于代码中我有很多疑惑,如果可以,我希望能有更方便的交流方式,比如邮件。我的邮箱是:lanxiaoao@163.com

@msftgits

msftgits commented Jul 24, 2020

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@goldscott

Copy link
Copy Markdown
Contributor

Hello @qepdclqu - thank you for reporting this and submitting your fixes. We will update our internal codebase with these changes and then move the changes over to github in our next patch release.

@yuxin-azrtos

Copy link
Copy Markdown
Contributor

@qepdclqu As @goldscott indicated, we are working on these fixes and will update the repo in our next release. Meanwhile I am closing this PR. Feel free to reach out to us if you have further questions.

@qepdclqu

qepdclqu commented Aug 9, 2020 via email

Copy link
Copy Markdown
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants