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

Re: [PATCH 3/5] x86/p2m: suppress audit_p2m hook when possible


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Tue, 10 Nov 2020 15:01:03 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.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-SenderADCheck; bh=d7owhZAVtvNdWNQB+tGU55PWMNYvbI7Tg2m0AILIEfM=; b=OvQOA3BaVj/R6uF7QhQjEMcWp0Wq5Zmg8iPS5lCGbeNtSBW5HqJec2WIbx9VkwJsLQoSDSIy8vjrSlS6oAwumObea7mzJ0haEl923WV4h/oGlc4JFspZTK6eZibZ8vwpp63MAmhFRoii0MQxHgw8XN+65u/Eq0pjTw7suBX3HI5DK0TE/Rvn6JOvuJ4Mou/ZmK8ZkMt5TLh3xfoAfOSW0Ik2QDmcsV+1p0nbf+DS5V+zS2qbUtRHNWuc6Reu0ArI0xii1asjt0j2a6A3qePishs7bAHzILX2/i4iAZmRRB5phbhjzLRpZOrmYJP23jc+AFykfI09WgE+xCKvaEH4MQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=OKDrMRU3QGqRyo0BurZIqKsX67wtz75p4glmdcKtCfEqQaGo8JOAQ09uB67caW9cFspcatRkgvPGugtysMi45A1IEDtk7BqWZAmQ9m9yhgJT6X5wbeB1cnJRMeWcqgo6enjsmiJGb//xD7NOMR69CDpd+L3PZ0D9zrA8zjARtuhqf87/cPzyTm1Ng7LlH9qwKKsoNDp9SV8rqk7Xmd9MOSETp8uBM4nvkgup/xOqRd4AdFZ/j8hnYfhvOQPKt0E7vUKDD7tfrQJOZnYrlrK1QMKQ23Lq/KKAA1vQ70gCelG0CcmwPmuesoc43WUsndvmRz6x2/u0/7CPf4Ql6iDrVg==
  • Authentication-results: esa6.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "Andrew Cooper" <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
  • Delivery-date: Tue, 10 Nov 2020 14:01:23 +0000
  • Ironport-sdr: uZTPLRCQA/o+O+5aULFK3NqufAa+I9YeBF+7vAn0ftWn36OoTHbizTHOlzhbESKstgP5zxGwcb Njq+OC4vDJxgPeDxUOSbR7/ZXAMDhcsMzcKQCSMwWKcs6/+jlvgCas7V3n3+amGfBHRGLJY7ED BV7WaDaWnkvkbXrZERuYNYEd6hJ091ri8jyYuD43QOAFfjc3y8sb64JEChRxeN6gnQdsg1j6Kh LvKBlczJpk6mS3b3mHpdtEKFxxFCQ/BeSPcS2yYk4mZBSwFT/l05aqOmYSUHR05XTEyD19RCYg e7s=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Tue, Nov 10, 2020 at 02:21:43PM +0100, Jan Beulich wrote:
> On 10.11.2020 12:30, Roger Pau Monné wrote:
> > On Wed, Oct 28, 2020 at 10:23:42AM +0100, Jan Beulich wrote:
> >> When P2M_AUDIT is false, it's unused, so instead of having a dangling
> >> NULL pointer sit there, omit the field altogether.
> >>
> >> Instead of adding "#if P2M_AUDIT && defined(CONFIG_HVM)" in even more
> >> places, fold the latter part right into the definition of P2M_AUDIT.
> >>
> >> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> > 
> > Acked-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> 
> Thanks. Since I see you keep replying to v1, are you aware of
> there being v2? For this particular patch there actually is a
> clang related fix in v2, which may be of interest to you.

Urg, no sorry. I was just going over my mail queue and didn't realize
there was a v2. Will take a look.

Roger.



 


Rackspace

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