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

Re: [PATCH v3 2/2] x86/mm: tidy XENMEM_{get,set}_pod_target handling


  • To: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Wed, 2 Feb 2022 16:29:37 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; 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=6KY+SS1IYWFi1LjRPIQdaTJLaih+KVVlSXmR3CEpSGw=; b=bgfdsaLp9YuQb7llvpyiri+q2+vV6H4Iq7RYJPAw/5MX6d3O3yroXXJT1V8YiEpj8F4YFqndQG0YbFMFM1j0Rql0u8/46xOZmSUptskeUDmqVABGzuHszcLACbaDzUIhE8R0mEpoJ5yO8S6/63LdCQk993xwy3VwB7Bg98Etx1b3BIG8qGcPSV97W8pcez9qNAUlvFWj1h9NT5kX96ooJ8aRHSTrwJNvxNnZ8psdv9dHLUHOi/MS124ABJRJdNou1aRQ5V2N6DixMwr2TxmVWnK/QeFRH/ATOddH38dC2ZIf203SMTW3uPZUAXkV9lss2LjKkJwXmEElrrdwPNBfTg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kz0xSPbLXva2fjgNbe5TSioDlLCHM1RyEGMEAlTn4huVewklL91Gx3/pzJlmQ6KKKM9w0o05eXoZF/rFm1GwagjqT2ntjjePFqK3Cv+DkMFdpyLkaKZhoTmBIubTNbkMe4CwA5VozRjrC5cQSbdEkMwCxR1rASypShNOb5rOfWBxpImN3UWftnbBF+n6d6x6XYthej3g0EjHd6PYT39zmk88ppie1RJd5TUX+1Emf4V5LPbzWR1VJSXmyAn6vg37Wq9bQFVas1i8ZXwZRrSoVM8zWQKKc3runHNn1ZeOZwzW3Y/GdSLiKiGeyt7aA6a7BlWpzxgBP0k/9lMOXYE9Xw==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>
  • Delivery-date: Wed, 02 Feb 2022 15:29:50 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 02.02.2022 16:14, Roger Pau Monné wrote:
> On Tue, Jan 04, 2022 at 10:41:53AM +0100, Jan Beulich wrote:
>> Do away with the "pod_target_out_unlock" label. In particular by folding
>> if()-s, the logic can be expressed with less code (and no goto-s) this
>> way.
>>
>> Limit scope of "p2m", constifying it at the same time.
> 
> Is this stale? I cannot find any reference to a p2m variable in the
> chunks below.

Indeed it is, leftover from rebasing over the introduction of
p2m_pod_get_mem_target() in what is now patch 1. Dropped.

Jan




 


Rackspace

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