-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
或许可以考虑将test-kernel独立成一个新的项目? #5
Comments
看上去这是一个好的思路,不过最近主维护者 @luojia65 似乎忙于其他事物 |
询问了一下,test-kernel 根据项目可能需要增补一些平台相关的测试。如果考虑独立出来的话,需要设计一下架构,看看如何便于扩展。 |
我觉得test-kernel只要能够测试SBI调用是否正确就好了,基本上应该可以做到平台无关的。而且为了能够在不同平台和不同的SBI实现上使用,我觉得也不应该把test-kernel做得太平台相关。 |
其实我觉得可以先把仓库建起来,把现在的代码扔到仓库里面去,后面我们再慢慢贡献代码。
…On 8/6/21 11:24 PM, Campbell He wrote:
询问了一下,test-kernel
根据项目可能需要增补一些平台相关的测试。如果考虑独立出来的话,需要设计一下架构,看看如何便于扩展。
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#5 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKQJSG33LPZVLY4F2JRFSGDT3P5BNANCNFSM5BBVBR5Q>.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email>.
|
这个其实没太必要,测试内核就那么点代码量,没必要占用组织的一个仓库。该组织的仓库基本上都应该为rustsbi和rustsbi的各个平台下的实现预留的。 |
这样做也可以,看后面项目如何发展了 |
test-kernel 的 test 部分现在已经挪动到 sbi-testing。我希望 test-kernel 是一个库的组合体,不知道是否满足你的期望? |
这个仓库看上去很不错。用库的形式,通过API允许测试者根据需要构建测试,这应该会是一个更好的形式。 |
注意到作为测试用例的test-kernel目前是作为该项目以及rustsbi-k210项目的一部分存在的。考虑到其作为测试用例而言对这两个sbi实现具有普适性,是否可以将test-kernel独立出来?这样还可以方便其他的SBI实现使用其作为测试用例。
The text was updated successfully, but these errors were encountered: