[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v5 4/6] x86/gdbstub: Clean up includes


  • To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Thu, 21 Apr 2022 15:09:00 +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:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=i62EqEUg1FfwSh0GvIbYWih43SAiLcTQYBloPkznT2E=; b=BEf/MG/9RnUWso7dhUA9qI9SLuz65l/QqoCdZU/LhhZDR4PWLcFnQ7eIq80/Mdv4k3ZSPoaRAL8BCxyDyD6hOaAvZ2vWuyl6J2GBQBNzMqOHTTmAKo34YJkY6WYQOc7l1e20vhXgYjEdsi+g+oYC5ALSdi3rnYX5r9BTLzuo61CkBLJE4qpZEILwnbu/98d0naWc0nLpNcLyqNeWZA2nXokJd/PinxfrWuh/ObYk9r1LOwXGjbxMmxP5xLyGqa1LQL6owrI+WOqbFqm5TCLI/CUGSr323M+GDhuGFAiGsE694Dz7XLXWopc17ZSGwfaKOFBBWMSI4A3Ma4GSe3Udzg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XtN0H0v6TMO9MkMGsIhcjRM3Lh5vsCyUQzZnxk342k7IW+tzsFeAko6g/90FqmLLWjPMPXFauG09rDBF7ZPmvlb+kgFpHrC2yQWp/k5uHbISOmIVdClKvMFhx7EQ1RQq346B1UzKwVyrSnlPMZbhDAGGAbuIQZmVGO0rnwCa9wpUe2b01kOEIKmfPT/F2eEp++EByEZ6DH6VSkdLu9obkjfGZ3lj5Nd9P5p0IgIs82Nc8WoJO9gKddDeTjwrv82I2IQSbIPsmWnt931X/Ra0wCI5QZlNUOwnH11zJxvDu8vQaDa9yDOsaRJJcpyVvI43pKDeF/RswYkDVFGc0D5KAg==
  • Authentication-results: dkim=none (message not signed) header.d=none;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: Thu, 21 Apr 2022 13:09:07 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 20.04.2022 16:13, Andrew Cooper wrote:
> common/gdbstub.c wants struct gdb_context but only gets it transitively
> through asm/debugger.h.  None of */gdbstub.c should include asm/debugger.h so
> include xen/gdbstub.h instead.
> 
> Forward declare struct cpu_user_regs in xen/gdbstub.h so it doesn't depend on
> the include order to compile.
> 
> x86/setup.c doesn't need xen/gdbstub.h at all, so drop it.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Acked-by: Jan Beulich <jbeulich@xxxxxxxx>




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.