ppc64 and Documentation/mic/mpssd/mpssd.c issues

2014-12-04 Thread Daniel Borkmann

Hi Caz,

I think commit ...

commit 8d49751580db804a02caf6a5b7cebe2ff26c0d7e
Author: Caz Yokoyama 
Date:   Thu Sep 5 16:42:39 2013 -0700

Sample Implementation of Intel MIC User Space Daemon.

... actually triggers a build error on my default config
ppc64 (I'm using net-next tree):

  HOSTCC  Documentation/mic/mpssd/mpssd.o
Documentation/mic/mpssd/mpssd.c:93: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:96: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:113: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:116: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:119: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:146: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:149: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:151: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:152: error: braced-group within expression 
allowed only inside a function
make[3]: *** [Documentation/mic/mpssd/mpssd.o] Error 1
make[2]: *** [Documentation/mic/mpssd] Error 2
make[1]: *** [Documentation/mic] Error 2
make: *** [vmlinux] Error 2

gcc --version
gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-11)

Cheers,
Daniel
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: ppc64 and Documentation/mic/mpssd/mpssd.c issues

2014-12-04 Thread Daniel Borkmann

On 12/04/2014 05:57 PM, Sudeep Dutt wrote:

On Thu, 2014-12-04 at 08:48 -0800, Yokoyama, Caz wrote:

Thank you for the report.
Have you tried to compile on x86_64? Which rhel do you use? Rhel6.X?
MIC card is expected to work with x86_64 host, not with ppc64. We have never 
compiled on ppc host while our primary development platform was rhel6.X.


Yep, I was using RHEL6 on ppc64 by chance to build an upstream kernel.

You might want to fix the Kconfig entry then to limit this to x86_64 only. ;)

Thanks,
Daniel


I am adding Peter Foley since he enabled the mpssd build by default
recently with commit adb19fb. Peter, is there a way to enable the mpssd
build only for x86_64?

Thanks,
Sudeep Dutt


Line 92 is the first use of htole16 and MIC_VRING_ENTRIES. Is there any change 
on them?

Thank you.
-caz, caz.yokoy...@intel.com, yokoy...@member.fsf.org


-Original Message-
From: Daniel Borkmann [mailto:dbork...@redhat.com]
Sent: Thursday, December 04, 2014 5:42 AM
To: Yokoyama, Caz
Cc: b...@kernel.crashing.org; linuxppc-dev@lists.ozlabs.org
Subject: ppc64 and Documentation/mic/mpssd/mpssd.c issues

Hi Caz,

I think commit ...

commit 8d49751580db804a02caf6a5b7cebe2ff26c0d7e
Author: Caz Yokoyama 
Date:   Thu Sep 5 16:42:39 2013 -0700

  Sample Implementation of Intel MIC User Space Daemon.

... actually triggers a build error on my default config
ppc64 (I'm using net-next tree):

HOSTCC  Documentation/mic/mpssd/mpssd.o
Documentation/mic/mpssd/mpssd.c:93: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:96: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:113: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:116: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:119: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:146: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:149: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:151: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:152: error: braced-group within expression 
allowed only inside a function
make[3]: *** [Documentation/mic/mpssd/mpssd.o] Error 1
make[2]: *** [Documentation/mic/mpssd] Error 2
make[1]: *** [Documentation/mic] Error 2
make: *** [vmlinux] Error 2

gcc --version
gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-11)

Cheers,
Daniel




___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: ppc64 and Documentation/mic/mpssd/mpssd.c issues

2014-12-04 Thread Benjamin Herrenschmidt
On Thu, 2014-12-04 at 16:48 +, Yokoyama, Caz wrote:
> Thank you for the report.
> Have you tried to compile on x86_64? Which rhel do you use? Rhel6.X?
> MIC card is expected to work with x86_64 host, not with ppc64. We have never 
> compiled on ppc host while our primary development platform was rhel6.X.
> 
> Line 92 is the first use of htole16 and MIC_VRING_ENTRIES. Is there any 
> change on them?

Then you should make sure your Kconfig/Makefile stanza only compiles it
on x86_64 instead of breaking all other architectures...

Ben.

> Thank you.
> -caz, caz.yokoy...@intel.com, yokoy...@member.fsf.org
> 
> 
> -Original Message-
> From: Daniel Borkmann [mailto:dbork...@redhat.com] 
> Sent: Thursday, December 04, 2014 5:42 AM
> To: Yokoyama, Caz
> Cc: b...@kernel.crashing.org; linuxppc-dev@lists.ozlabs.org
> Subject: ppc64 and Documentation/mic/mpssd/mpssd.c issues
> 
> Hi Caz,
> 
> I think commit ...
> 
> commit 8d49751580db804a02caf6a5b7cebe2ff26c0d7e
> Author: Caz Yokoyama 
> Date:   Thu Sep 5 16:42:39 2013 -0700
> 
>  Sample Implementation of Intel MIC User Space Daemon.
> 
> ... actually triggers a build error on my default config
> ppc64 (I'm using net-next tree):
> 
>HOSTCC  Documentation/mic/mpssd/mpssd.o
> Documentation/mic/mpssd/mpssd.c:93: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:96: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:113: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:116: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:119: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:146: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:149: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:151: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:152: error: braced-group within expression 
> allowed only inside a function
> make[3]: *** [Documentation/mic/mpssd/mpssd.o] Error 1
> make[2]: *** [Documentation/mic/mpssd] Error 2
> make[1]: *** [Documentation/mic] Error 2
> make: *** [vmlinux] Error 2
> 
> gcc --version
> gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-11)
> 
> Cheers,
> Daniel


___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

RE: ppc64 and Documentation/mic/mpssd/mpssd.c issues

2014-12-04 Thread Sudeep Dutt
On Thu, 2014-12-04 at 08:48 -0800, Yokoyama, Caz wrote:
> Thank you for the report.
> Have you tried to compile on x86_64? Which rhel do you use? Rhel6.X?
> MIC card is expected to work with x86_64 host, not with ppc64. We have never 
> compiled on ppc host while our primary development platform was rhel6.X.
> 

I am adding Peter Foley since he enabled the mpssd build by default
recently with commit adb19fb. Peter, is there a way to enable the mpssd
build only for x86_64?

Thanks,
Sudeep Dutt

> Line 92 is the first use of htole16 and MIC_VRING_ENTRIES. Is there any 
> change on them?
> 
> Thank you.
> -caz, caz.yokoy...@intel.com, yokoy...@member.fsf.org
> 
> 
> -Original Message-
> From: Daniel Borkmann [mailto:dbork...@redhat.com] 
> Sent: Thursday, December 04, 2014 5:42 AM
> To: Yokoyama, Caz
> Cc: b...@kernel.crashing.org; linuxppc-dev@lists.ozlabs.org
> Subject: ppc64 and Documentation/mic/mpssd/mpssd.c issues
> 
> Hi Caz,
> 
> I think commit ...
> 
> commit 8d49751580db804a02caf6a5b7cebe2ff26c0d7e
> Author: Caz Yokoyama 
> Date:   Thu Sep 5 16:42:39 2013 -0700
> 
>  Sample Implementation of Intel MIC User Space Daemon.
> 
> ... actually triggers a build error on my default config
> ppc64 (I'm using net-next tree):
> 
>HOSTCC  Documentation/mic/mpssd/mpssd.o
> Documentation/mic/mpssd/mpssd.c:93: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:96: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:113: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:116: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:119: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:146: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:149: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:151: error: braced-group within expression 
> allowed only inside a function
> Documentation/mic/mpssd/mpssd.c:152: error: braced-group within expression 
> allowed only inside a function
> make[3]: *** [Documentation/mic/mpssd/mpssd.o] Error 1
> make[2]: *** [Documentation/mic/mpssd] Error 2
> make[1]: *** [Documentation/mic] Error 2
> make: *** [vmlinux] Error 2
> 
> gcc --version
> gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-11)
> 
> Cheers,
> Daniel


___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

RE: ppc64 and Documentation/mic/mpssd/mpssd.c issues

2014-12-04 Thread Yokoyama, Caz
Thank you for the report.
Have you tried to compile on x86_64? Which rhel do you use? Rhel6.X?
MIC card is expected to work with x86_64 host, not with ppc64. We have never 
compiled on ppc host while our primary development platform was rhel6.X.

Line 92 is the first use of htole16 and MIC_VRING_ENTRIES. Is there any change 
on them?

Thank you.
-caz, caz.yokoy...@intel.com, yokoy...@member.fsf.org


-Original Message-
From: Daniel Borkmann [mailto:dbork...@redhat.com] 
Sent: Thursday, December 04, 2014 5:42 AM
To: Yokoyama, Caz
Cc: b...@kernel.crashing.org; linuxppc-dev@lists.ozlabs.org
Subject: ppc64 and Documentation/mic/mpssd/mpssd.c issues

Hi Caz,

I think commit ...

commit 8d49751580db804a02caf6a5b7cebe2ff26c0d7e
Author: Caz Yokoyama 
Date:   Thu Sep 5 16:42:39 2013 -0700

 Sample Implementation of Intel MIC User Space Daemon.

... actually triggers a build error on my default config
ppc64 (I'm using net-next tree):

   HOSTCC  Documentation/mic/mpssd/mpssd.o
Documentation/mic/mpssd/mpssd.c:93: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:96: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:113: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:116: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:119: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:146: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:149: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:151: error: braced-group within expression 
allowed only inside a function
Documentation/mic/mpssd/mpssd.c:152: error: braced-group within expression 
allowed only inside a function
make[3]: *** [Documentation/mic/mpssd/mpssd.o] Error 1
make[2]: *** [Documentation/mic/mpssd] Error 2
make[1]: *** [Documentation/mic] Error 2
make: *** [vmlinux] Error 2

gcc --version
gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-11)

Cheers,
Daniel
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: ppc64 and Documentation/mic/mpssd/mpssd.c issues

2014-12-04 Thread Peter Foley
On Thu, Dec 4, 2014 at 11:57 AM, Sudeep Dutt  wrote:
> On Thu, 2014-12-04 at 08:48 -0800, Yokoyama, Caz wrote:
>> Thank you for the report.
>> Have you tried to compile on x86_64? Which rhel do you use? Rhel6.X?
>> MIC card is expected to work with x86_64 host, not with ppc64. We have never 
>> compiled on ppc host while our primary development platform was rhel6.X.
>>
>
> I am adding Peter Foley since he enabled the mpssd build by default
> recently with commit adb19fb. Peter, is there a way to enable the mpssd
> build only for x86_64?

Change hostprogs-y to hostprogs-$(CONFIG_X86_64) in the mpssd Makefile.
That will restrict it to x86_64 only.
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev