From mboxrd@z Thu Jan 1 00:00:00 1970 From: Radim =?utf-8?B?S3LEjW3DocWZ?= Subject: Re: [PATCH RFC 1/2] KVM: x86: generalize guest_cpuid_has_ helpers Date: Fri, 4 Aug 2017 22:48:27 +0200 Message-ID: <20170804204827.GB2119@potion> References: <20170802204147.3586-1-rkrcmar@redhat.com> <20170802204147.3586-2-rkrcmar@redhat.com> <3924d878-c8f1-06f7-1051-19864708211e@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-kernel@vger.kernel.org, kvm@vger.kernel.org, David Hildenbrand To: Paolo Bonzini Return-path: Content-Disposition: inline In-Reply-To: <3924d878-c8f1-06f7-1051-19864708211e@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org 2017-08-04 17:40+0200, Paolo Bonzini: > Wow, I didn't expect the compiler to be able to inline all of this and > even do BUILD_BUG_ON()s on array lookups. Maybe change inline to > __always_inline just to be safe? > > If anybody complains we can make it just a BUG. Good point, i386 wanted to have non-inlined guest_cpuid_get_register(), but I hope that __always_inline is going to be enough for all. Thanks.