From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Hildenbrand Subject: Re: [PATCH v2 1/5] KVM: rename API for requests to match bit operations Date: Mon, 27 Feb 2017 10:54:09 +0100 Message-ID: <69dff1a5-6af4-ce1e-bb5c-3e3adb1ee6ae@redhat.com> References: <20170224195002.28030-1-rkrcmar@redhat.com> <20170224195002.28030-2-rkrcmar@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: Paolo Bonzini , Andrew Jones , Marc Zyngier , Christian Borntraeger , Cornelia Huck , James Hogan , Paul Mackerras , Christoffer Dall To: =?UTF-8?B?UmFkaW0gS3LEjW3DocWZ?= , linux-kernel@vger.kernel.org, kvm@vger.kernel.org Return-path: In-Reply-To: <20170224195002.28030-2-rkrcmar@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org Am 24.02.2017 um 20:49 schrieb Radim Krčmář: > kvm_make_request was a wrapper that added barriers to set_bit and > kvm_check_request did the same for test_bit and check_bit, but the name > was not very obvious. > > The renaming: > kvm_request_set <- kvm_make_request > kvm_request_test_and_clear <- kvm_check_request > What about kvm_req_set() / kvm_req_test_and_clear() Matches the request bit definitions (KVM_REQ_*) and saves a couple of characters. Anyhow Reviewed-by: David Hildenbrand -- Thanks, David