On Thu, Aug 04, 2016 at 01:27:37PM -0600, Jens Axboe wrote: > On 08/04/2016 11:50 AM, kbuild test robot wrote: > >Hi Mike, > > > >[auto build test ERROR on linus/master] > >[also build test ERROR on next-20160804] > >[cannot apply to v4.7] > >[if your patch is applied to the wrong git tree, please drop us a note to help improve the system] > > > >url: https://github.com/0day-ci/linux/commits/Mike-Christie/mm-block-convert-rw_page-users-to-bio-op-use/20160805-012041 > >config: i386-tinyconfig (attached as .config) > >compiler: gcc-6 (Debian 6.1.1-9) 6.1.1 20160705 > >reproduce: > > # save the attached .config to linux build tree > > make ARCH=i386 > > > >All errors (new ones prefixed by >>): > > > > mm/filemap.c: In function 'page_endio': > >>>mm/filemap.c:892:7: error: implicit declaration of function 'op_is_write' [-Werror=implicit-function-declaration] > > if (!op_is_write(op)) { > > ^~~~~~~~~~~ > > Mike, how about moving op_is_write() outside of CONFIG_BLOCK protection > in fs.h, and making the REQ_OP_* enum generally available as well? That > should fix it. Give this is being spread all over the kernel way outside the block layer and IO path, shouldn't this have some kind of namespace component to the name? i.e "req_op_is_write()"? Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
