Thanks for your advice, I will do this better later :) -----邮件原件----- 发件人: Christian Borntraeger [mailto:borntraeger@xxxxxxxxxx] 发送时间: 2021年2月4日 16:42 收件人: Cornelia Huck <cohuck@xxxxxxxxxx>; zhengyongjun <zhengyongjun3@xxxxxxxxxx> 抄送: kvm@xxxxxxxxxxxxxxx; linux-s390@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; frankja@xxxxxxxxxxxxx; david@xxxxxxxxxx; imbrenda@xxxxxxxxxxxxx; hca@xxxxxxxxxxxxx; gor@xxxxxxxxxxxxx 主题: Re: [PATCH -next] KVM: s390: Return the correct errno code 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.
