Hi Sage,
After merging the ceph tree, today's linux-next build (x86_64
allmodconfig) failed like this:
In file included from include/linux/kernel.h:14:0,
from include/linux/cache.h:4,
from include/linux/time.h:4,
from include/linux/stat.h:18,
from include/linux/module.h:10,
from fs/ceph/file.c:3:
fs/ceph/file.c: In function 'ceph_sync_read':
fs/ceph/file.c:421:21: error: 'struct kiocb' has no member named 'ki_left'
(unsigned)iocb->ki_left,
^
include/linux/dynamic_debug.h:79:10: note: in definition of macro 'dynamic_pr_debug'
##__VA_ARGS__); \
^
include/linux/ceph/ceph_debug.h:17:2: note: in expansion of macro 'pr_debug'
pr_debug("%.*s %12.12s:%-4d : " fmt, \
^
fs/ceph/file.c:420:2: note: in expansion of macro 'dout'
dout("sync_read on file %p %llu~%u %s\n", file, off,
^
fs/ceph/file.c:430:17: error: 'struct kiocb' has no member named 'ki_left'
off + iocb->ki_left);
^
fs/ceph/file.c:436:25: error: 'struct iov_iter' has no member named 'iov'
void __user *data = i->iov[0].iov_base + i->iov_offset;
^
fs/ceph/file.c:437:18: error: 'struct iov_iter' has no member named 'iov'
size_t len = i->iov[0].iov_len - i->iov_offset;
^
fs/ceph/file.c:458:20: error: 'struct kiocb' has no member named 'ki_left'
size_t len = iocb->ki_left;
^
fs/ceph/file.c:471:26: error: 'struct iov_iter' has no member named 'iov'
void __user *data = i->iov[0].iov_base
^
In file included from include/linux/cache.h:4:0,
from include/linux/time.h:4,
from include/linux/stat.h:18,
from include/linux/module.h:10,
from fs/ceph/file.c:3:
fs/ceph/file.c:473:14: error: 'struct iov_iter' has no member named 'iov'
l = min(i->iov[0].iov_len - i->iov_offset,
^
include/linux/kernel.h:670:9: note: in definition of macro 'min'
typeof(x) _min1 = (x); \
^
fs/ceph/file.c:473:14: error: 'struct iov_iter' has no member named 'iov'
l = min(i->iov[0].iov_len - i->iov_offset,
^
include/linux/kernel.h:670:21: note: in definition of macro 'min'
typeof(x) _min1 = (x); \
^
include/linux/kernel.h:672:17: warning: comparison of distinct pointer types lacks a cast [enabled by default]
(void) (&_min1 == &_min2); \
^
fs/ceph/file.c:473:9: note: in expansion of macro 'min'
l = min(i->iov[0].iov_len - i->iov_offset,
^
fs/ceph/file.c:496:7: error: 'struct kiocb' has no member named 'ki_left'
iocb->ki_left -= ret;
^
fs/ceph/file.c: In function 'ceph_sync_direct_write':
fs/ceph/file.c:588:24: error: 'struct iov_iter' has no member named 'iov'
void __user *data = i.iov->iov_base + i.iov_offset;
^
fs/ceph/file.c:589:14: error: 'struct iov_iter' has no member named 'iov'
u64 len = i.iov->iov_len - i.iov_offset;
^
fs/ceph/file.c: In function 'ceph_aio_read':
fs/ceph/file.c:839:7: error: 'struct kiocb' has no member named 'ki_left'
iocb->ki_left = len;
^
fs/ceph/file.c:870:8: error: 'struct kiocb' has no member named 'ki_left'
iocb->ki_left) {
^
Caused by commit e6c9af8b8f11 ("ceph: implement readv/preadv for sync
operation"). ki_left was removed by commit 73a7075e3f6e ("aio: Kill
aio_rw_vect_retry()") during the merge window ... basing new work for
v3.13 on v3.11 is a bit problematic.
I have used the ceph tree from next-20130925 for today.
--
Cheers,
Stephen Rothwell [email protected]
Sorry for that, i notice the commit 73a7075e3f6e.But i misread it.
I'll modify this as soon as possible.
Thanks!
Jianpeng Ma
>Hi Sage,
>
>After merging the ceph tree, today's linux-next build (x86_64
>allmodconfig) failed like this:
>
>In file included from include/linux/kernel.h:14:0,
> from include/linux/cache.h:4,
> from include/linux/time.h:4,
> from include/linux/stat.h:18,
> from include/linux/module.h:10,
> from fs/ceph/file.c:3:
>fs/ceph/file.c: In function 'ceph_sync_read':
>fs/ceph/file.c:421:21: error: 'struct kiocb' has no member named 'ki_left'
> (unsigned)iocb->ki_left,
> ^
>include/linux/dynamic_debug.h:79:10: note: in definition of macro 'dynamic_pr_debug'
> ##__VA_ARGS__); \
> ^
>include/linux/ceph/ceph_debug.h:17:2: note: in expansion of macro 'pr_debug'
> pr_debug("%.*s %12.12s:%-4d : " fmt, \
> ^
>fs/ceph/file.c:420:2: note: in expansion of macro 'dout'
> dout("sync_read on file %p %llu~%u %s\n", file, off,
> ^
>fs/ceph/file.c:430:17: error: 'struct kiocb' has no member named 'ki_left'
> off + iocb->ki_left);
> ^
>fs/ceph/file.c:436:25: error: 'struct iov_iter' has no member named 'iov'
> void __user *data = i->iov[0].iov_base + i->iov_offset;
> ^
>fs/ceph/file.c:437:18: error: 'struct iov_iter' has no member named 'iov'
> size_t len = i->iov[0].iov_len - i->iov_offset;
> ^
>fs/ceph/file.c:458:20: error: 'struct kiocb' has no member named 'ki_left'
> size_t len = iocb->ki_left;
> ^
>fs/ceph/file.c:471:26: error: 'struct iov_iter' has no member named 'iov'
> void __user *data = i->iov[0].iov_base
> ^
>In file included from include/linux/cache.h:4:0,
> from include/linux/time.h:4,
> from include/linux/stat.h:18,
> from include/linux/module.h:10,
> from fs/ceph/file.c:3:
>fs/ceph/file.c:473:14: error: 'struct iov_iter' has no member named 'iov'
> l = min(i->iov[0].iov_len - i->iov_offset,
> ^
>include/linux/kernel.h:670:9: note: in definition of macro 'min'
> typeof(x) _min1 = (x); \
> ^
>fs/ceph/file.c:473:14: error: 'struct iov_iter' has no member named 'iov'
> l = min(i->iov[0].iov_len - i->iov_offset,
> ^
>include/linux/kernel.h:670:21: note: in definition of macro 'min'
> typeof(x) _min1 = (x); \
> ^
>include/linux/kernel.h:672:17: warning: comparison of distinct pointer types lacks a cast [enabled by default]
> (void) (&_min1 == &_min2); \
> ^
>fs/ceph/file.c:473:9: note: in expansion of macro 'min'
> l = min(i->iov[0].iov_len - i->iov_offset,
> ^
>fs/ceph/file.c:496:7: error: 'struct kiocb' has no member named 'ki_left'
> iocb->ki_left -= ret;
> ^
>fs/ceph/file.c: In function 'ceph_sync_direct_write':
>fs/ceph/file.c:588:24: error: 'struct iov_iter' has no member named 'iov'
> void __user *data = i.iov->iov_base + i.iov_offset;
> ^
>fs/ceph/file.c:589:14: error: 'struct iov_iter' has no member named 'iov'
> u64 len = i.iov->iov_len - i.iov_offset;
> ^
>fs/ceph/file.c: In function 'ceph_aio_read':
>fs/ceph/file.c:839:7: error: 'struct kiocb' has no member named 'ki_left'
> iocb->ki_left = len;
> ^
>fs/ceph/file.c:870:8: error: 'struct kiocb' has no member named 'ki_left'
> iocb->ki_left) {
> ^
>
>Caused by commit e6c9af8b8f11 ("ceph: implement readv/preadv for sync
>operation"). ki_left was removed by commit 73a7075e3f6e ("aio: Kill
>aio_rw_vect_retry()") during the merge window ... basing new work for
>v3.13 on v3.11 is a bit problematic.
>
>I have used the ceph tree from next-20130925 for today.
>
>--
>Cheers,
>Stephen Rothwell [email protected]
>????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay? ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?
On Thu, 26 Sep 2013, majianpeng wrote:
> Sorry for that, i notice the commit 73a7075e3f6e.But i misread it.
> I'll modify this as soon as possible.
Thanks! For now I'll drop this patch and rebase the queue against
3.12-rc2.
Thanks, Stephen!
sage
>
> Thanks!
> Jianpeng Ma
> >Hi Sage,
> >
> >After merging the ceph tree, today's linux-next build (x86_64
> >allmodconfig) failed like this:
> >
> >In file included from include/linux/kernel.h:14:0,
> > from include/linux/cache.h:4,
> > from include/linux/time.h:4,
> > from include/linux/stat.h:18,
> > from include/linux/module.h:10,
> > from fs/ceph/file.c:3:
> >fs/ceph/file.c: In function 'ceph_sync_read':
> >fs/ceph/file.c:421:21: error: 'struct kiocb' has no member named 'ki_left'
> > (unsigned)iocb->ki_left,
> > ^
> >include/linux/dynamic_debug.h:79:10: note: in definition of macro 'dynamic_pr_debug'
> > ##__VA_ARGS__); \
> > ^
> >include/linux/ceph/ceph_debug.h:17:2: note: in expansion of macro 'pr_debug'
> > pr_debug("%.*s %12.12s:%-4d : " fmt, \
> > ^
> >fs/ceph/file.c:420:2: note: in expansion of macro 'dout'
> > dout("sync_read on file %p %llu~%u %s\n", file, off,
> > ^
> >fs/ceph/file.c:430:17: error: 'struct kiocb' has no member named 'ki_left'
> > off + iocb->ki_left);
> > ^
> >fs/ceph/file.c:436:25: error: 'struct iov_iter' has no member named 'iov'
> > void __user *data = i->iov[0].iov_base + i->iov_offset;
> > ^
> >fs/ceph/file.c:437:18: error: 'struct iov_iter' has no member named 'iov'
> > size_t len = i->iov[0].iov_len - i->iov_offset;
> > ^
> >fs/ceph/file.c:458:20: error: 'struct kiocb' has no member named 'ki_left'
> > size_t len = iocb->ki_left;
> > ^
> >fs/ceph/file.c:471:26: error: 'struct iov_iter' has no member named 'iov'
> > void __user *data = i->iov[0].iov_base
> > ^
> >In file included from include/linux/cache.h:4:0,
> > from include/linux/time.h:4,
> > from include/linux/stat.h:18,
> > from include/linux/module.h:10,
> > from fs/ceph/file.c:3:
> >fs/ceph/file.c:473:14: error: 'struct iov_iter' has no member named 'iov'
> > l = min(i->iov[0].iov_len - i->iov_offset,
> > ^
> >include/linux/kernel.h:670:9: note: in definition of macro 'min'
> > typeof(x) _min1 = (x); \
> > ^
> >fs/ceph/file.c:473:14: error: 'struct iov_iter' has no member named 'iov'
> > l = min(i->iov[0].iov_len - i->iov_offset,
> > ^
> >include/linux/kernel.h:670:21: note: in definition of macro 'min'
> > typeof(x) _min1 = (x); \
> > ^
> >include/linux/kernel.h:672:17: warning: comparison of distinct pointer types lacks a cast [enabled by default]
> > (void) (&_min1 == &_min2); \
> > ^
> >fs/ceph/file.c:473:9: note: in expansion of macro 'min'
> > l = min(i->iov[0].iov_len - i->iov_offset,
> > ^
> >fs/ceph/file.c:496:7: error: 'struct kiocb' has no member named 'ki_left'
> > iocb->ki_left -= ret;
> > ^
> >fs/ceph/file.c: In function 'ceph_sync_direct_write':
> >fs/ceph/file.c:588:24: error: 'struct iov_iter' has no member named 'iov'
> > void __user *data = i.iov->iov_base + i.iov_offset;
> > ^
> >fs/ceph/file.c:589:14: error: 'struct iov_iter' has no member named 'iov'
> > u64 len = i.iov->iov_len - i.iov_offset;
> > ^
> >fs/ceph/file.c: In function 'ceph_aio_read':
> >fs/ceph/file.c:839:7: error: 'struct kiocb' has no member named 'ki_left'
> > iocb->ki_left = len;
> > ^
> >fs/ceph/file.c:870:8: error: 'struct kiocb' has no member named 'ki_left'
> > iocb->ki_left) {
> > ^
> >
> >Caused by commit e6c9af8b8f11 ("ceph: implement readv/preadv for sync
> >operation"). ki_left was removed by commit 73a7075e3f6e ("aio: Kill
> >aio_rw_vect_retry()") during the merge window ... basing new work for
> >v3.13 on v3.11 is a bit problematic.
> >
> >I have used the ceph tree from next-20130925 for today.
> >
> >--
> >Cheers,
> >Stephen Rothwell [email protected]
> >