[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 10/12] x86/pv: Move x86/trace.c to x86/pv/trace.c
- To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Tue, 21 Sep 2021 17:58:25 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=YzVDtu6c47NQj1352kpAYcG/QTW8cdOFMazIOozwIr8=; b=nCI3sqMZI7ozvplt6TQrZgqjjgjF2nSHCsYzRZPhZVzAr84T+K/iQ7ey7WES5yw8daNlR8Wu8R4efc+qj+odrqoSOb5f5YK0HR701RpOtkEbj+1VehHizZGrHzSVs9hdg4QQY9Z3H1il6+n+DGhpO15y80dlpL7avvja35ix5ldQjB/u/gl6K56PO5zssuPx7P0HG4wLWiVtngDWe0rmDZkfC+jNkFNAc7v9+DAw7eB+nZm+lK4TW8+V24kzXwAVmHdY7EiMo+N1xPTfKf3GS77SMgwUEuuB7t35Nb5TuhcmObkCfkxhOTcnBJOBwDICL5oW+j1wyW3kt/9E7Hwnkg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=YH4huxZxQfVk5Foj7iR4zVc58bFjMfBrOv7E3HytfVqzcgnqvzYtvbXJVaCg9AJOVDHMQ9GiHcTXCk9uHbG9tpd6XnjdMfu7ioWGniT1687xb4Qcusvp/XE3o9OCIqxWVOyDn1DY1oK87/N8neD2VjWdHxMt10CBPZ8dzn7pSkeolKNsWPO4f4S3fctHewVr3VB8HNqKOnLB1Qf2SYbIuprb8ke3Qt33RB4M3T/yKo9ZJc/5Bnlnbc+57lK7MiiVAOAWUKVPxF48nYEK+lHPOdsl90oOKZhHRY7GQDU5Ss86qUGureogM4n+nLzMOKvoqxNmvSBfJoymUqoD4ywRvQ==
- Authentication-results: lists.xenproject.org; dkim=none (message not signed) header.d=none;lists.xenproject.org; dmarc=none action=none header.from=suse.com;
- Cc: Roger Pau Monné <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Tue, 21 Sep 2021 15:58:39 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 20.09.2021 19:25, Andrew Cooper wrote:
> This entire file is pv-only, and not excluded from the build by
> CONFIG_TRACEBUFFER. Move it into the pv/ directory, build it conditionally,
> and drop unused includes.
>
> Also move the contents of asm/trace.h to asm/pv/trace.h to avoid the functions
> being declared across the entire hypervisor.
>
> One caller in fixup_page_fault() is effectively PV only, but is not subject to
> dead code elimination. Add an additional IS_ENABLED(CONFIG_PV) to keep the
> build happy.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
|