Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751558AbdG0Hpn (ORCPT ); Thu, 27 Jul 2017 03:45:43 -0400 Received: from smtp-out6.electric.net ([192.162.217.186]:56975 "EHLO smtp-out6.electric.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750992AbdG0Hpm (ORCPT ); Thu, 27 Jul 2017 03:45:42 -0400 From: David Laight To: "'Brijesh Singh'" , "H. Peter Anvin" , Arnd Bergmann CC: "linux-kernel@vger.kernel.org" , "x86@kernel.org" , "linux-efi@vger.kernel.org" , "linuxppc-dev@lists.ozlabs.org" , "kvm@vger.kernel.org" , Fenghua Yu , Matt Fleming , David Howells , Paul Mackerras , Christoph Lameter , Jonathan Corbet , =?utf-8?B?UmFkaW0gS3JjbcOhcg==?= , Piotr Luc , Ingo Molnar , Dave Airlie , Borislav Petkov , Tom Lendacky , Kees Cook , "Konrad Rzeszutek Wilk" , Reza Arbab , Andy Lutomirski , "Thomas Gleixner" , Laura Abbott , Tony Luck , "Ard.Biesheuvel@zytor.com" Subject: RE: [RFC Part1 PATCH v3 13/17] x86/io: Unroll string I/O when SEV is active Thread-Topic: [RFC Part1 PATCH v3 13/17] x86/io: Unroll string I/O when SEV is active Thread-Index: AQHTBMr+maW+IBZqtEeA2Vfkk/FXraJkTNGggAIzVfz//+/MAIAAC0AAgADTATA= Date: Thu, 27 Jul 2017 07:45:25 +0000 Message-ID: <063D6719AE5E284EB5DD2968C1650D6DD00436A7@AcuExch.aculab.com> References: <20170724190757.11278-1-brijesh.singh@amd.com> <20170724190757.11278-14-brijesh.singh@amd.com> <063D6719AE5E284EB5DD2968C1650D6DD003FB85@AcuExch.aculab.com> <201707261927.v6QJR228008075@mail.zytor.com> <589d65a4-eb09-bae9-e8b4-a2d78ca6b509@amd.com> In-Reply-To: <589d65a4-eb09-bae9-e8b4-a2d78ca6b509@amd.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.202.99.200] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-Outbound-IP: 156.67.243.126 X-Env-From: David.Laight@ACULAB.COM X-Proto: esmtps X-Revdns: X-HELO: AcuExch.aculab.com X-TLS: TLSv1:AES128-SHA:128 X-Authenticated_ID: X-PolicySMART: 3396946, 3397078 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id v6R7jlQa029698 Content-Length: 481 Lines: 16 From: Brijesh Singh > Sent: 26 July 2017 21:07 ... > I am not sure if I understand your concern. > > Are you commenting on amount of code duplication ? If so, I can certainly improve > and use the similar macro used into header file to generate the functions body. If you are careful the real functions could expand the inline functions that get used when SEV is compiled out. Oh, if you are looking at this, can you fix memcpy_to_io() so that it is never 'rep movsb'? David