On 04.02.21 09:32, Cornelia Huck wrote: > On Thu, 4 Feb 2021 16:05:23 +0800 > Zheng Yongjun <zhengyongjun3@xxxxxxxxxx> wrote: > >> When valloc failed, should return ENOMEM rather than ENOBUF. >> >> Signed-off-by: Zheng Yongjun <zhengyongjun3@xxxxxxxxxx> >> --- >> arch/s390/kvm/interrupt.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c >> index 2f177298c663..6b7acc27cfa2 100644 >> --- a/arch/s390/kvm/interrupt.c >> +++ b/arch/s390/kvm/interrupt.c >> @@ -2252,7 +2252,7 @@ static int get_all_floating_irqs(struct kvm *kvm, u8 __user *usrbuf, u64 len) >> */ >> buf = vzalloc(len); >> if (!buf) >> - return -ENOBUFS; >> + return -ENOMEM; >> >> max_irqs = len / sizeof(struct kvm_s390_irq); >> > > This breaks a user space interface (see the comment right above the > vzalloc). Right. Please do not send (generated?) patches without looking at the code that you are patching.
