Received: by 2002:a25:c205:0:0:0:0:0 with SMTP id s5csp2433653ybf; Mon, 2 Mar 2020 08:32:12 -0800 (PST) X-Google-Smtp-Source: ADFU+vvo8pmE9owA1OfmJW2Kam1QbBgO8c3m0NgudlUpsj7WaEE2/tMtomYDhC8cA41sDIb4LROd X-Received: by 2002:aca:43c1:: with SMTP id q184mr225341oia.116.1583166731898; Mon, 02 Mar 2020 08:32:11 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1583166731; cv=none; d=google.com; s=arc-20160816; b=fAwabp2pDjndQT0dqcV10YN0N6YGNQOpnb2EqWRXO2UHw/eqdfms6nWd8voJpAymTT 3Yvxn58EP/5ezd2DwYFky+FhaRjUAo1yvjDnySdToA2PpIwimPxkbPpj+1AZ4CSYwvkA OEAGjMtqVtHDmUI2ByP0CFyRfl6EIvU2deBzXUVTDrpBgaXG2VtunmkO70Xsx33JguI8 Rbz3h9Sx6AfWYLr2mE7S89aR+rX5Me+kUf8J5IbOs/J/IiXGob/xBIB1cJJwoCW/RZh8 ziROymvfE29a/8MxlgxEzQx+vC65StV3kwSqZ0loPbvlETyZ4YTKL/x020WmIRI6as+c R/xw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature; bh=1x0ZM1xTpaIkqmKsVKdQPlGDF0qsj8hnKJSTjUcj/Aw=; b=zFFVNZTkiN4P8EW6F+nsP9bDLQ56NmiX0gWtWGKRdgozJnRrOqX4r0gROSFRogZkle E6bUwCX1SuWeVATbCplQiPOM2tylv/MxT0Z27Sbj3Yo2ImIRu/ETsCILyRyn33wYLLxa jlv+MsPOy5hm7TIbTY3aSP44UV+a6LEc1lvPaJk/os9JVa7gUO9g5i+L7zPsQL5yBeaU QgpU+04KwW8DNSg07GiN5d2rAvWmrI78IYw6eWHfWhZd4PchHeUY/1uva928gka219Gw FNXr4hyEdWRTGaLxa2rJarptxgf0izhdPUQG0dEYODQM7WqOJUnmwBV4oG73lQTZuYy9 bSKA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=fQHBlnME; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id r2si2691760otp.200.2020.03.02.08.32.00; Mon, 02 Mar 2020 08:32:11 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=fQHBlnME; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727202AbgCBQbe (ORCPT + 99 others); Mon, 2 Mar 2020 11:31:34 -0500 Received: from us-smtp-delivery-1.mimecast.com ([205.139.110.120]:31623 "EHLO us-smtp-1.mimecast.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727085AbgCBQbe (ORCPT ); Mon, 2 Mar 2020 11:31:34 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1583166693; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=1x0ZM1xTpaIkqmKsVKdQPlGDF0qsj8hnKJSTjUcj/Aw=; b=fQHBlnMEnRTtk57rFCbGRNAJeb95rlHRUmHhVDb2JnxLPROTFI86Wy5Xqm+kasmIIU+fFd D8aCD1SzRF8ygVgZbiDyBFo99EooI91l5ZC7HOFO6FVUMBlIHx3sVQOORYyM1M2bHS4ddd BjT0gU7IF4lkZ4F8YxOoKduimAtRqys= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-438-YN0MDv7TNfKWpEwlP2DjoA-1; Mon, 02 Mar 2020 11:31:32 -0500 X-MC-Unique: YN0MDv7TNfKWpEwlP2DjoA-1 Received: by mail-wr1-f69.google.com with SMTP id d9so6016688wrv.21 for ; Mon, 02 Mar 2020 08:31:31 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=1x0ZM1xTpaIkqmKsVKdQPlGDF0qsj8hnKJSTjUcj/Aw=; b=iLs3x2hgOZGineuX6CQMVEpLn8nK6RxImFuvKdzRGXfh9AbrbVH4ktK3+Ys478+SJ3 mqy6/GuEDvQDU11PWkut1Lff2AcEIEM/JnZP8atPQOlIGUxB+UFaGO5u08o6O/Im4/mo pT3SipNvw4ItHQw6nMbFo2BquSH1nXNoKbWJKzL9nb/y4GgFOLrfOThUIByoj4WdXVP7 rN4vtDp7e9utkQJCslCBIyrYILtD080m+7xWwjy9MIk9sjdeQn9YW5fKyMvwCSwgIyTL 2I5OTwIGF0fwK6CuAAe4hOtri2wtFg4/m9USICePbeb7y+mziqpf70LVxSrfYPXTdYGj X6iA== X-Gm-Message-State: ANhLgQ1IKmEmd99SEfy15+K93kOlCxcv3WGAMtW+pGdOl0G55OIaBEd5 TqQXi4LDMzhdYeC4KBg1sm2sC7SVFqicrZ2awIarnyDQoq9S7zcs0w2qNSfWfKDJIpC68ocfYZN mEJ1zlOCSnzJXqcb6p/HiKmye X-Received: by 2002:a05:6000:100d:: with SMTP id a13mr397865wrx.330.1583166690988; Mon, 02 Mar 2020 08:31:30 -0800 (PST) X-Received: by 2002:a05:6000:100d:: with SMTP id a13mr397846wrx.330.1583166690776; Mon, 02 Mar 2020 08:31:30 -0800 (PST) Received: from [192.168.178.40] ([151.30.85.6]) by smtp.gmail.com with ESMTPSA id s22sm15544619wmc.16.2020.03.02.08.31.29 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 02 Mar 2020 08:31:30 -0800 (PST) Subject: Re: [PATCH v2] KVM: X86: deprecate obsolete KVM_GET_CPUID2 ioctl To: Vitaly Kuznetsov , linmiaohe , rkrcmar@redhat.com, sean.j.christopherson@intel.com, wanpengli@tencent.com, jmattson@google.com, joro@8bytes.org, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, hpa@zytor.com Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, x86@kernel.org References: <1582773688-4956-1-git-send-email-linmiaohe@huawei.com> <87ftewi7of.fsf@vitty.brq.redhat.com> From: Paolo Bonzini Message-ID: <6c78b2c8-98b2-a517-4426-511505ff9018@redhat.com> Date: Mon, 2 Mar 2020 17:31:29 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 MIME-Version: 1.0 In-Reply-To: <87ftewi7of.fsf@vitty.brq.redhat.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 27/02/20 12:36, Vitaly Kuznetsov wrote: >> - if (copy_to_user(cpuid_arg, &cpuid, sizeof(cpuid))) >> - goto out; >> - r = 0; >> + r = -EINVAL; >> break; >> } > Braces are not really needed not but all other cases in the switch have > it so let's leave them here too. > We can remove the case altogether. Paolo