Backport subject : Userspace pointer masking and tagged address ABI

[fangyu0809]

fixed: #335

自测试：
1, pointer_masking self test:
Testing available PMLEN values
ok 1 PMLEN=0 PR_GET_TAGGED_ADDR_CTRL
ok 2 PMLEN=0 constraint
ok 3 PMLEN=0 validity
ok 4 PMLEN=1 PR_GET_TAGGED_ADDR_CTRL
ok 5 PMLEN=1 constraint
ok 6 PMLEN=1 validity
ok 7 PMLEN=2 PR_GET_TAGGED_ADDR_CTRL
ok 8 PMLEN=2 constraint
ok 9 PMLEN=2 validity
...
Testing tagged address ABI
ok 59 PMLEN=0 tagged address ABI
ok 60 PMLEN=7 tagged address ABI
ok 61 PMLEN=16 tagged address ABI
Totals: pass:61 fail:0 xfail:0 xpass:0 skip:0 error:0

2,kvm-get-reg-list smnpm
smnpm: PASS
3,kvm-get-reg-list ssnpm
ssnpm: PASS

[unicornx]

请详细提供测试的过程，一来我这里需要验证一下，另外可以考虑以后集成到回归测试中。
最好是基于 qemu 的测试方法。谢谢

[unicornx]

1st review:

通用问题：
commit 的信息和上游原始 commit 不一致，主要体现在你摘取的 commit 看上去不是主线的 commit，譬如主线都会带有 LKML link 信息，但是你丢掉了，我怀疑你是拿的 LKML 的 patch，不是 linus 仓库 master 上的 commit。请检查。

具体到 commit 有以下几个问题

• 对 "riscv: Add support for the tagged address ABI" 这个 commit，我发现原补丁中下面这个你没有pick，是何原因？

+#ifdef CONFIG_RISCV_ISA_SUPM
+#define mm_untag_mask mm_untag_mask
+static inline unsigned long mm_untag_mask(struct mm_struct *mm)
+{
+	return -1UL >> mm->context.pmlen;
+}
+#endif
+

如果 backpor 时不是简单的 pick 而是的确有需要改动，需要在 commit message 中做出说明，具体位置就是在

    mainline inclusion
    from Linux 6.13-rc1
    commit 29eedc7d1587f42f33ae209be45c89c424ee9c00
    category: feature
    bugzilla: https://github.com/RVCK-Project/rvck/issues/335
	
	如果基于原上游补丁有自己的修改，请在这里添加修改的原因和修改方案说明。
    
    --------------------------------

• 对 "riscv: selftests: Add a pointer masking test" 这个 commit。 同理，不是简单的 pick 有改动，请补充说明
  我是指

+RISCV_SUBTARGETS ?= abi hwprobe vector mm sse cfi

• 对 "riscv: add compile-time test into is_compat_task()" 这个 commit， 缺少原补丁中对 arch/riscv/include/asm/processor.h 的改动，请补充说明。

[fangyu0809]

1st review:

通用问题： commit 的信息和上游原始 commit 不一致，主要体现在你摘取的 commit 看上去不是主线的 commit，譬如主线都会带有 LKML link 信息，但是你丢掉了，我怀疑你是拿的 LKML 的 patch，不是 linus 仓库 master 上的 commit。请检查。

这里的确是从mail list拿的提交，需要修改一下

具体到 commit 有以下几个问题

• 对 "riscv: Add support for the tagged address ABI" 这个 commit，我发现原补丁中下面这个你没有pick，是何原因？

+#ifdef CONFIG_RISCV_ISA_SUPM
+#define mm_untag_mask mm_untag_mask
+static inline unsigned long mm_untag_mask(struct mm_struct *mm)
+{
+	return -1UL >> mm->context.pmlen;
+}
+#endif
+

mm_untag_mask()：本地父提交已存在，来自 97644be

如果 backpor 时不是简单的 pick 而是的确有需要改动，需要在 commit message 中做出说明，具体位置就是在

    mainline inclusion
    from Linux 6.13-rc1
    commit 29eedc7d1587f42f33ae209be45c89c424ee9c00
    category: feature
    bugzilla: https://github.com/RVCK-Project/rvck/issues/335
	
	如果基于原上游补丁有自己的修改，请在这里添加修改的原因和修改方案说明。
    
    --------------------------------

• 对 "riscv: selftests: Add a pointer masking test" 这个 commit。 同理，不是简单的 pick 有改动，请补充说明
  我是指

+RISCV_SUBTARGETS ?= abi hwprobe vector mm sse cfi

RISCV_SUBTARGETS：只加入 abi，保留 rvck 的 sse cfi，不引入本地不存在的 upstream context sigreturn。

• 对 "riscv: add compile-time test into is_compat_task()" 这个 commit， 缺少原补丁中对 arch/riscv/include/asm/processor.h 的改动，请补充说明。

processor.h：rvck 已有后续 stable 修复 f5f235b / upstream 2116988，对应代码已被简化，upstream 4c0b 的两处修改目标不存在。

[fangyu0809]

请详细提供测试的过程，一来我这里需要验证一下，另外可以考虑以后集成到回归测试中。 最好是基于 qemu 的测试方法。谢谢

重新编译 selftests
- RISC-V ABI selftest：
make -C tools/testing/selftests/riscv/abi CC=riscv64-unknown-linux-gnu-gcc
- KVM get-reg-list selftest，静态链接：
make -C tools/testing/selftests/kvm ARCH=riscv CC=riscv64-unknown-linux-gnu-gcc CROSS_COMPILE=riscv64-unknown-linux-gnu- LDFLAGS=-static .../get-reg-list
重新打包 initramfs
- 把最新的：
- pointer_masking
- get-reg-list
系统起来后，进行pointer_masking self test和kvm-get-reg-list测试

[github-actions]

---

开始测试 log: https://github.com/RVCK-Project/rvck/actions/runs/29914895629

参数解析结果

args	value
repository	RVCK-Project/rvck
head ref	pull/336/head
base ref	rvck-6.6
LAVA repo	RVCK-Project/lavaci
LAVA hardware	['qemu']
LAVA Testcase path	lava-testcases/common-test/ltp/ltp.yaml
need run job	kunit-test,kernel-build,check-patch,lava-trigger

---

测试完成

详细结果:

check	result
kunit-test	success
kernel-build	success
check-patch	success
lava-trigger-qemu	success
lava-trigger-sg2042	skipped
lava-trigger-k1	skipped
lava-trigger-lpi4a	skipped

Kunit Test Result

[11:17:59] Testing complete. Ran 482 tests: passed: 465, skipped: 17

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/29914895629/attempts/1/#summary-88906368756

Kernel Build Result

• Kernel build succeeded: RVCK-Project/rvck/336_29914895629_1/

• c85d3e01ca9cef9fbece5328eb84a874 /srv/guix_result/a433d82ffae99eba11d74fdfaad7bbd091fbd92a/Image

• 1a9e81d2740c580ce9f1c00e80fee357 /root/initramfs.img

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/29914895629/attempts/1/#summary-88906368671

Check Patch Result

Total Errors	0
Total Warnings	19

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/29914895629/attempts/1/#summary-88906368794

LAVA Check (qemu)

args	value
testcase_repo	RVCK-Project/lavaci
lava_template	lava-job-template/qemu/qemu-ltp.yaml
testcase_path	lava-testcases/common-test/ltp/ltp.yaml
kernel_download_url	http://10.30.190.110/openEuler-RISC-V/RVCK/OERV-RVCI/RVCK-Project/rvck/336_29914895629_1/Image
initramfs_download_url	http://10.30.190.110/openEuler-RISC-V/RVCK/OERV-RVCI/RVCK-Project/rvck/336_29914895629_1/initramfs.img
rootfs_download_url	https://fast-mirror.isrc.ac.cn/openeuler-sig-riscv/openEuler-RISC-V/RVCK/openEuler24.03-LTS-SP1/openeuler-rootfs.img.zst
testcase_ref	main
testitem_name	RVCK-Project_rvck_pull_request_target_336__common-test_qemu

result: Lava check done!

• lava log: https://lava.oerv.ac.cn/scheduler/job/3533

• lava result count: [pass]: 10

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/29914895629/attempts/1/#summary-88917056390

[unicornx]

2nd review

针对 “riscv: Add support for the tagged address ABI” 这笔提交，你的 commit message 中的如下描述有个小问题需要改一下

Backport note:
RVCK already contains mm_untag_mask() in
arch/riscv/include/asm/mmu_context.h from commit 97644be6352a
(riscv/shstk: If needed allocate a new shadow stack on clone), so this
backport does not add that hunk again and only adds the remaining
pmlen/context locking changes.

在引用 rvck-6.6 分支上的补丁时，只要引用标题即可，不要引用 commit 的 hashcode。原因是 rvck-6.6 分支每个月会 rebase，导致该分支上的所有 commit 的 hashcode 会发生改变（这是目前 rvck-6.6 分支的管理方式决定的，和主线的 master 不同，主线的 master 上 commit 永远不变的）。所以我们引用时如果用了 hashcode，反而会对以后的人阅读造成困惑，而 commit 的 title 是不变的。所以可以改成如下方式：

Backport note:
RVCK already contains mm_untag_mask() in
arch/riscv/include/asm/mmu_context.h from commit 
"riscv/shstk: If needed allocate a new shadow stack on clone", so this
backport does not add that hunk again and only adds the remaining
pmlen/context locking changes.

我仔细看了一下修改历史，实际上这个问题我不清楚是不是当初 rvck 上合入 "riscv/shstk: If needed allocate a new shadow stack on clone" 的一个 bug，照理当时就应该已经 back port "riscv: Add support for the tagged address ABI" 了。怎么能直接改上游的补丁呢。

其他的 LGTM。

我再找时间测试一下。

[unicornx]

请详细提供测试的过程，一来我这里需要验证一下，另外可以考虑以后集成到回归测试中。 最好是基于 qemu 的测试方法。谢谢

重新编译 selftests - RISC-V ABI selftest： make -C tools/testing/selftests/riscv/abi CC=riscv64-unknown-linux-gnu-gcc - KVM get-reg-list selftest，静态链接： make -C tools/testing/selftests/kvm ARCH=riscv CC=riscv64-unknown-linux-gnu-gcc CROSS_COMPILE=riscv64-unknown-linux-gnu- LDFLAGS=-static .../get-reg-list 重新打包 initramfs - 把最新的： - pointer_masking - get-reg-list 系统起来后，进行pointer_masking self test和kvm-get-reg-list测试

我构建了 pointer_masking 和 get-reg-list。并将他们复制到 qemu riscv64 上运行

pointer_masking 可以正常 pass，见附件
pointer_masking.log

但 get-reg-list 运行失败，

$ ./get-reg-list --config=smnpm
1..0 # SKIP - /dev/kvm not available (errno: 2)
$ ./get-reg-list --config=ssnpm
1..0 # SKIP - /dev/kvm not available (errno: 2)

两个问题：

1. 我看你在 PR 中写的是 kvm-get-reg-list smnpm 和 kvm-get-reg-list ssnpm 是不是写错了？生成的测试程序是 get-reg-list，没有 kvm-get-reg-list
2. 是不是在 QEMU 上无法测试 KVM 相关的 case? 请问你是在什么设备上测试的?

[fangyu0809]

请详细提供测试的过程，一来我这里需要验证一下，另外可以考虑以后集成到回归测试中。 最好是基于 qemu 的测试方法。谢谢

重新编译 selftests - RISC-V ABI selftest： make -C tools/testing/selftests/riscv/abi CC=riscv64-unknown-linux-gnu-gcc - KVM get-reg-list selftest，静态链接： make -C tools/testing/selftests/kvm ARCH=riscv CC=riscv64-unknown-linux-gnu-gcc CROSS_COMPILE=riscv64-unknown-linux-gnu- LDFLAGS=-static .../get-reg-list 重新打包 initramfs - 把最新的： - pointer_masking - get-reg-list 系统起来后，进行pointer_masking self test和kvm-get-reg-list测试

我构建了 pointer_masking 和 get-reg-list。并将他们复制到 qemu riscv64 上运行

pointer_masking 可以正常 pass，见附件 pointer_masking.log

但 get-reg-list 运行失败，

$ ./get-reg-list --config=smnpm
1..0 # SKIP - /dev/kvm not available (errno: 2)
$ ./get-reg-list --config=ssnpm
1..0 # SKIP - /dev/kvm not available (errno: 2)

两个问题：

1. 我看你在 PR 中写的是 kvm-get-reg-list smnpm 和 kvm-get-reg-list ssnpm 是不是写错了？生成的测试程序是 get-reg-list，没有 kvm-get-reg-list

是get-reg-list

2. 是不是在 QEMU 上无法测试 KVM 相关的 case? 请问你是在什么设备上测试的?

在Qemu上测试的，编译内核时打开KVM的config

[unicornx]

在Qemu上测试的，编译内核时打开KVM的config

我用的 rvck 的 defconfig，CONFIG_KVM=y

请问你用的 QEMU 是什么版本，以及启动 QEMU 的选项参数是什么？

[fangyu0809]

在Qemu上测试的，编译内核时打开KVM的config

我用的 rvck 的 defconfig，CONFIG_KVM=y

请问你用的 QEMU 是什么版本，以及启动 QEMU 的选项参数是什么？

帮忙把你启动qemu的参数，还有你内核启动的log贴一下，我这边check一下。

[unicornx]

发现一个问题，make clean 时 tools/testing/selftests/riscv/abi/pointer_masking 清不掉，请看看上游是否有补丁 fix 这个清理？

[unicornx]

在Qemu上测试的，编译内核时打开KVM的config

我用的 rvck 的 defconfig，CONFIG_KVM=y
请问你用的 QEMU 是什么版本，以及启动 QEMU 的选项参数是什么？

帮忙把你启动qemu的参数，还有你内核启动的log贴一下，我这边check一下。

看上去我这里暂时找不到可以支持 kvm 的 rv 硬件作为 host 测试 kvm 的特性，kvm 的测试就靠你先测试吧，你先把我提的几个问题先解决了吧。
p.s. rvck-6.6 更新了，记得 rebase 一下先

[fangyu0809]

发现一个问题，make clean 时 tools/testing/selftests/riscv/abi/pointer_masking 清不掉，请看看上游是否有补丁 fix 这个清理？

检索了一下，没看到针对这个问题的上游patch。这个问题也不是pointer_masking引入的，建议不在当前pr跟踪解决了。

[github-actions]

---

开始测试 log: https://github.com/RVCK-Project/rvck/actions/runs/30353319661

参数解析结果

args	value
repository	RVCK-Project/rvck
head ref	pull/336/head
base ref	rvck-6.6
LAVA repo	RVCK-Project/lavaci
LAVA hardware	['qemu']
LAVA Testcase path	lava-testcases/common-test/ltp/ltp.yaml
need run job	kunit-test,kernel-build,check-patch,lava-trigger

---

测试完成

详细结果:

check	result
kunit-test	success
kernel-build	success
check-patch	success
lava-trigger-qemu	failure
lava-trigger-sg2042	skipped
lava-trigger-k1	skipped
lava-trigger-lpi4a	skipped

Kunit Test Result

[11:09:36] Testing complete. Ran 482 tests: passed: 465, skipped: 17

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/30353319661/attempts/1/#summary-90255709216

Kernel Build Result

• Kernel build succeeded: RVCK-Project/rvck/336_30353319661_1/

• e9f2b7645560126b14c10df9cee9f5d6 /srv/guix_result/5ee9685d967ff1dc0c91b31af09322e9a4eef68b/Image

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/30353319661/attempts/1/#summary-90255709208

Check Patch Result

Total Errors	0
Total Warnings	19

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/30353319661/attempts/1/#summary-90255709215

LAVA Check (qemu)

---

lava 执行失败.

[fangyu0809]

/check

---

开始测试 log: https://github.com/RVCK-Project/rvck/actions/runs/30794244938

参数解析结果

args	value
repository	RVCK-Project/rvck
head ref	pull/336/head
base ref	rvck-6.6
LAVA repo	RVCK-Project/lavaci
LAVA hardware	['qemu']
LAVA Testcase path	lava-testcases/common-test/ltp/ltp.yaml
need run job	kunit-test,kernel-build,check-patch,lava-trigger

---

测试完成

详细结果:

check	result
kunit-test	success
kernel-build	success
check-patch	success
lava-trigger-qemu	failure
lava-trigger-sg2042	skipped
lava-trigger-k1	skipped
lava-trigger-lpi4a	skipped

Kunit Test Result

[07:41:03] Testing complete. Ran 482 tests: passed: 465, skipped: 17

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/30794244938/attempts/1/#summary-91624136946

Kernel Build Result

• Kernel build succeeded: RVCK-Project/rvck/336_30794244938_1/

• d7864482436e8b657d7e77e3a98fd4ad /srv/guix_result/5ee9685d967ff1dc0c91b31af09322e9a4eef68b/Image

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/30794244938/attempts/1/#summary-91624136972

Check Patch Result

Total Errors	0
Total Warnings	19

• detail log: https://github.com/RVCK-Project/rvck/actions/runs/30794244938/attempts/1/#summary-91624136940

LAVA Check (qemu)

---

lava 执行失败.

[sterling-teng]

8月份的rvck-6.6滚动基线很快会推送，该pr赶不上了，届时rebase后继续审阅吧。