Skip to content

[PW_SID:1094939] [1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM#1940

Open
linux-riscv-bot wants to merge 2 commits into
workflow__riscv__fixesfrom
pw1094939
Open

[PW_SID:1094939] [1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM#1940
linux-riscv-bot wants to merge 2 commits into
workflow__riscv__fixesfrom
pw1094939

Conversation

@linux-riscv-bot
Copy link
Copy Markdown

PR for series 1094939 applied to workflow__riscv__fixes

Name: [1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM
URL: https://patchwork.kernel.org/project/linux-riscv/list/?series=1094939
Version: 1

kvm_riscv_vcpu_pmu_snapshot_set_shmem() returned -ENOMEM from the
SBI extension handler, which caused kvm_riscv_vcpu_sbi_ecall() to
abort KVM_RUN and surface the error to userspace instead of
ompleting the ECALL with a negative SBI error in a0.
Use SBI_ERR_FAILURE and the normal retdata path, matching other PMU
handlers and kvm_sbi_ext_pmu_handler comment.

Fixes: c2f41dd ("RISC-V: KVM: Implement SBI PMU Snapshot feature")
Cc: stable@vger.kernel.org
Signed-off-by: Osama Abdelkader <osama.abdelkader@gmail.com>
Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
kvm_riscv_vcpu_pmu_event_info() returned -ENOMEM from the
SBI extension handler, which caused kvm_riscv_vcpu_sbi_ecall()
to abort KVM_RUN and surface the error to userspace instead of
completing the ECALL with a negative SBI error in a0.
Use SBI_ERR_FAILURE and the normal retdata path, matching other PMU
handlers and kvm_sbi_ext_pmu_handler comment.

Fixes: e309fd1 ("RISC-V: KVM: Implement get event info function")
Cc: stable@vger.kernel.org
Signed-off-by: Osama Abdelkader <osama.abdelkader@gmail.com>
Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
build-rv32-defconfig
Desc: Builds riscv32 defconfig
Duration: 139.09 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
build-rv64-clang-allmodconfig
Desc: Builds riscv64 allmodconfig with Clang, and checks for errors and added warnings
Duration: 1043.84 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
build-rv64-gcc-allmodconfig
Desc: Builds riscv64 allmodconfig with GCC, and checks for errors and added warnings
Duration: 1426.44 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
build-rv64-nommu-k210-defconfig
Desc: Builds riscv64 defconfig with NOMMU for K210
Duration: 25.99 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
build-rv64-nommu-k210-virt
Desc: Builds riscv64 defconfig with NOMMU for the virt platform
Duration: 26.91 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
checkpatch
Desc: Runs checkpatch.pl on the patch
Duration: 0.79 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
dtb-warn-rv64
Desc: Checks for Device Tree warnings/errors
Duration: 86.95 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
header-inline
Desc: Detects static functions without inline keyword in header files
Duration: 0.24 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
kdoc
Desc: Detects for kdoc errors
Duration: 0.82 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
module-param
Desc: Detect module_param changes
Duration: 0.25 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
verify-fixes
Desc: Verifies that the Fixes: tags exist
Duration: 0.29 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[1/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_snapshot_set_shmem OOM"
verify-signedoff
Desc: Verifies that Signed-off-by: tags are correct
Duration: 0.30 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
build-rv32-defconfig
Desc: Builds riscv32 defconfig
Duration: 137.75 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
build-rv64-clang-allmodconfig
Desc: Builds riscv64 allmodconfig with Clang, and checks for errors and added warnings
Duration: 1040.07 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
build-rv64-gcc-allmodconfig
Desc: Builds riscv64 allmodconfig with GCC, and checks for errors and added warnings
Duration: 1425.53 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
build-rv64-nommu-k210-defconfig
Desc: Builds riscv64 defconfig with NOMMU for K210
Duration: 25.56 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
build-rv64-nommu-k210-virt
Desc: Builds riscv64 defconfig with NOMMU for the virt platform
Duration: 27.06 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
checkpatch
Desc: Runs checkpatch.pl on the patch
Duration: 0.82 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
dtb-warn-rv64
Desc: Checks for Device Tree warnings/errors
Duration: 87.02 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
header-inline
Desc: Detects static functions without inline keyword in header files
Duration: 0.24 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
kdoc
Desc: Detects for kdoc errors
Duration: 0.89 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
module-param
Desc: Detect module_param changes
Duration: 0.26 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
verify-fixes
Desc: Verifies that the Fixes: tags exist
Duration: 0.28 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 2: "[2/2] riscv: kvm: return SBI_ERR_FAILURE for pmu_event_info OOM"
verify-signedoff
Desc: Verifies that Signed-off-by: tags are correct
Duration: 0.31 seconds
Result: PASS

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.

2 participants