Re: [PATCH v4 4/9] staging:lustre: merge socklnd_lib-linux.h into socklnd.h

2015-06-25 Thread Geert Uytterhoeven
On Thu, Jun 25, 2015 at 3:33 AM, Guenter Roeck li...@roeck-us.net wrote: I have not tested it, but I think the following may fix the problem while avoiding any include problems. Since pt_regs is used in the file, one could argue that it should be declared. Indeed. I tried that, but... --

RE: [PATCH v4 4/9] staging:lustre: merge socklnd_lib-linux.h into socklnd.h

2015-06-25 Thread Simmons, James A.
diff --git a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h index 53275f9..7125eb9 100644 --- a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h +++ b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd.h @@ -25,16 +25,40 @@

RE: [PATCH v4 4/9] staging:lustre: merge socklnd_lib-linux.h into socklnd.h

2015-06-25 Thread Simmons, James A.
On Thu, Jun 25, 2015 at 3:33 AM, Guenter Roeck li...@roeck-us.net wrote: I have not tested it, but I think the following may fix the problem while avoiding any include problems. Since pt_regs is used in the file, one could argue that it should be declared. Indeed. I tried that, but... --

Re: [PATCH v4 4/9] staging:lustre: merge socklnd_lib-linux.h into socklnd.h

2015-06-24 Thread Geert Uytterhoeven
Hi James, On Thu, Jun 11, 2015 at 9:18 PM, James Simmons jsimm...@infradead.org wrote: From: John L. Hammond john.hamm...@intel.com Originally socklnd_lib-linux.h contained linux specific wrappers and defines but since the linux kernel is the only supported platform now we can merge what

Re: [PATCH v4 4/9] staging:lustre: merge socklnd_lib-linux.h into socklnd.h

2015-06-24 Thread Guenter Roeck
On Wed, Jun 24, 2015 at 02:37:51PM +0200, Geert Uytterhoeven wrote: Hi James, On Thu, Jun 11, 2015 at 9:18 PM, James Simmons jsimm...@infradead.org wrote: From: John L. Hammond john.hamm...@intel.com Originally socklnd_lib-linux.h contained linux specific wrappers and defines but since