Re: [Xen-devel] [PATCH for-4.9] xsm: fix clang 3.5 build after c47d1d

2017-04-10 Thread Daniel De Graaf
On 04/10/2017 04:41 AM, Roger Pau Monne wrote: On Fri, Apr 07, 2017 at 08:54:59AM -0600, Jan Beulich wrote: On 07.04.17 at 16:34, wrote: The changes introduced on c47d1d broke the clang build due to undefined references to __xsm_action_mismatch_detected, because clang hasn't optimized the code

Re: [Xen-devel] [PATCH for-4.9] xsm: fix clang 3.5 build after c47d1d

2017-04-10 Thread Roger Pau Monne
On Fri, Apr 07, 2017 at 08:54:59AM -0600, Jan Beulich wrote: > >>> On 07.04.17 at 16:34, wrote: > > The changes introduced on c47d1d broke the clang build due to undefined > > references to __xsm_action_mismatch_detected, because clang hasn't optimized > > the code properly. > > This starts to be

Re: [Xen-devel] [PATCH for-4.9] xsm: fix clang 3.5 build after c47d1d

2017-04-10 Thread Jan Beulich
>>> On 09.04.17 at 21:01, wrote: > On 04/07/2017 03:54 PM, Jan Beulich wrote: > On 07.04.17 at 16:34, wrote: >>> The changes introduced on c47d1d broke the clang build due to undefined >>> references to __xsm_action_mismatch_detected, because clang hasn't optimized >>> the code properly. >>

Re: [Xen-devel] [PATCH for-4.9] xsm: fix clang 3.5 build after c47d1d

2017-04-09 Thread Julien Grall
Hi Roger, On 04/07/2017 03:34 PM, Roger Pau Monne wrote: The changes introduced on c47d1d broke the clang build due to undefined references to __xsm_action_mismatch_detected, because clang hasn't optimized the code properly. The following patch allows the clang build to work again, while keeping

Re: [Xen-devel] [PATCH for-4.9] xsm: fix clang 3.5 build after c47d1d

2017-04-09 Thread Julien Grall
Hi, On 04/07/2017 03:54 PM, Jan Beulich wrote: On 07.04.17 at 16:34, wrote: >> The changes introduced on c47d1d broke the clang build due to undefined >> references to __xsm_action_mismatch_detected, because clang hasn't optimized >> the code properly. > > This starts to become annoying. T

Re: [Xen-devel] [PATCH for-4.9] xsm: fix clang 3.5 build after c47d1d

2017-04-07 Thread Jan Beulich
>>> On 07.04.17 at 16:34, wrote: > The changes introduced on c47d1d broke the clang build due to undefined > references to __xsm_action_mismatch_detected, because clang hasn't optimized > the code properly. This starts to become annoying. > --- a/xen/include/xsm/dummy.h > +++ b/xen/include/xsm/d

Re: [Xen-devel] [PATCH for-4.9] xsm: fix clang 3.5 build after c47d1d

2017-04-07 Thread M A Young
On Fri, 7 Apr 2017, Roger Pau Monne wrote: ... > +return xsm_default_action(XSM_TARGET, current->domain, d); > case XEN_ALTP2M_external: > -a = XSM_DM_PRIV; > +return xsm_default_action(XSM_DM_PRIV, current->domain, d); > break; > case XEN_ALTP2M_limited:

[Xen-devel] [PATCH for-4.9] xsm: fix clang 3.5 build after c47d1d

2017-04-07 Thread Roger Pau Monne
The changes introduced on c47d1d broke the clang build due to undefined references to __xsm_action_mismatch_detected, because clang hasn't optimized the code properly. The following patch allows the clang build to work again, while keeping the same functionality. Signed-off-by: Roger Pau Monné --