Re: [PATCH] fs: dax: do not build on ARC or SH

2015-07-13 Thread Matthew Wilcox
On Mon, Jul 13, 2015 at 02:57:10PM +0300, Boaz Harrosh wrote: > I do not understand why we need to call copy_user_page here at all? > the destination is kmap_atomic() so it must be there right? also the > destination is the cow-to page so surly it is not yet mapped to user-space > mapping. > > the

Re: [PATCH] fs: dax: do not build on ARC or SH

2015-07-13 Thread Boaz Harrosh
On 07/13/2015 12:52 PM, Geert Uytterhoeven wrote: > From: Arnd Bergmann > > The DAX implementation relies on the architecture to provide a working > copy_user_page() function, as reported by Michael Ellerman's kisskb > build bot: > > fs/dax.c: error: implicit declaration of function 'copy_user_p

[PATCH] fs: dax: do not build on ARC or SH

2015-07-13 Thread Geert Uytterhoeven
From: Arnd Bergmann The DAX implementation relies on the architecture to provide a working copy_user_page() function, as reported by Michael Ellerman's kisskb build bot: fs/dax.c: error: implicit declaration of function 'copy_user_page' [-Werror=implicit-function-declaration]: => 266:2 We alr

Re: fs: dax: do not build on ARC or SH

2015-02-27 Thread Paul Bolle
On Mon, 2015-02-23 at 12:29 +0100, Arnd Bergmann wrote: > The DAX implementation relies on the architecture to provide a working > copy_user_page() function, as reported by Michael Ellerman's kisskb > build bot: > > fs/dax.c: error: implicit declaration of function 'copy_user_page' > [-Werror=imp

Re: fs: dax: do not build on ARC or SH

2015-02-23 Thread Guenter Roeck
On Mon, Feb 23, 2015 at 01:58:47PM -0800, Christoph Hellwig wrote: > On Mon, Feb 23, 2015 at 01:56:34PM -0800, Guenter Roeck wrote: > > Guess there a philosophical difference in opinion if the architecture code > > should (have to) provide copy_user_page() or not outside the architecture > > code i

Re: fs: dax: do not build on ARC or SH

2015-02-23 Thread Christoph Hellwig
On Mon, Feb 23, 2015 at 01:56:34PM -0800, Guenter Roeck wrote: > Guess there a philosophical difference in opinion if the architecture code > should (have to) provide copy_user_page() or not outside the architecture > code itself. After all, fs/dax.c _is_ the only user of this function outside > th

Re: fs: dax: do not build on ARC or SH

2015-02-23 Thread Guenter Roeck
On Mon, Feb 23, 2015 at 01:40:25PM -0500, Matthew Wilcox wrote: > On Mon, Feb 23, 2015 at 12:29:45PM +0100, Arnd Bergmann wrote: > > The DAX implementation relies on the architecture to provide a working > > copy_user_page() function, as reported by Michael Ellerman's kisskb > > build bot: > > > >

Re: fs: dax: do not build on ARC or SH

2015-02-23 Thread Arnd Bergmann
On Monday 23 February 2015 13:40:25 Matthew Wilcox wrote: > On Mon, Feb 23, 2015 at 12:29:45PM +0100, Arnd Bergmann wrote: > > The DAX implementation relies on the architecture to provide a working > > copy_user_page() function, as reported by Michael Ellerman's kisskb > > build bot: > > > > fs/da

Re: fs: dax: do not build on ARC or SH

2015-02-23 Thread Matthew Wilcox
On Mon, Feb 23, 2015 at 12:29:45PM +0100, Arnd Bergmann wrote: > The DAX implementation relies on the architecture to provide a working > copy_user_page() function, as reported by Michael Ellerman's kisskb > build bot: > > fs/dax.c: error: implicit declaration of function 'copy_user_page' > [-Wer

fs: dax: do not build on ARC or SH

2015-02-23 Thread Arnd Bergmann
The DAX implementation relies on the architecture to provide a working copy_user_page() function, as reported by Michael Ellerman's kisskb build bot: fs/dax.c: error: implicit declaration of function 'copy_user_page' [-Werror=implicit-function-declaration]: => 266:2 We already have a list of ar