Re: [PATCH 2/2] iov_iter: use bvec iterator to implement iterate_bvec()

From: kbuild test robot
Date: Sat Oct 10 2015 - 11:34:42 EST

Next message: Christoffer Dall: "Re: [PATCH v2 11/20] kvm: rename pfn_t to kvm_pfn_t"
Previous message: Russell King - ARM Linux: "Re: [PATCH] sched_clock: add data pointer argument to read callback"
In reply to: Ming Lei: "[PATCH 2/2] iov_iter: use bvec iterator to implement iterate_bvec()"
Next in thread: Ming Lei: "Re: [PATCH 2/2] iov_iter: use bvec iterator to implement iterate_bvec()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Ming,

[auto build test ERROR on v4.3-rc4 -- if it's inappropriate base, please ignore]

config: i386-randconfig-s1-201540 (attached as .config)
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All error/warnings (new ones prefixed by >>):

lib/iov_iter.c: In function 'copy_to_iter':
>> lib/iov_iter.c:61:19: error: storage size of '__bi' isn't known
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:386:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:61:25: error: storage size of '__start' isn't known
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:386:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:66:2: error: implicit declaration of function 'for_each_bvec' [-Werror=implicit-function-declaration]
for_each_bvec(__v, __p, __bi, __start) { \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:386:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:66:41: error: expected ';' before '{' token
for_each_bvec(__v, __p, __bi, __start) { \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:386:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:71:2: error: 'else' without a previous 'if'
else \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:386:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:61:25: warning: unused variable '__start' [-Wunused-variable]
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:386:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:61:19: warning: unused variable '__bi' [-Wunused-variable]
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:386:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
lib/iov_iter.c: In function 'copy_from_iter':
>> lib/iov_iter.c:61:19: error: storage size of '__bi' isn't known
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:407:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:61:25: error: storage size of '__start' isn't known
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:407:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:66:41: error: expected ';' before '{' token
for_each_bvec(__v, __p, __bi, __start) { \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:407:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:71:2: error: 'else' without a previous 'if'
else \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:407:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:61:25: warning: unused variable '__start' [-Wunused-variable]
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:407:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:61:19: warning: unused variable '__bi' [-Wunused-variable]
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:407:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
lib/iov_iter.c: In function 'copy_from_iter_nocache':
>> lib/iov_iter.c:61:19: error: storage size of '__bi' isn't known
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:428:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:61:25: error: storage size of '__start' isn't known
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:428:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:66:41: error: expected ';' before '{' token
for_each_bvec(__v, __p, __bi, __start) { \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:428:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:71:2: error: 'else' without a previous 'if'
else \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:428:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:61:25: warning: unused variable '__start' [-Wunused-variable]
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:428:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
>> lib/iov_iter.c:61:19: warning: unused variable '__bi' [-Wunused-variable]
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:428:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^
lib/iov_iter.c: In function 'iov_iter_zero':
>> lib/iov_iter.c:61:19: error: storage size of '__bi' isn't known
struct bvec_iter __bi, __start; \
^
lib/iov_iter.c:97:3: note: in expansion of macro 'iterate_bvec'
iterate_bvec(i, n, v, bvec, skip, (B)) \
^
lib/iov_iter.c:474:2: note: in expansion of macro 'iterate_and_advance'
iterate_and_advance(i, bytes, v,
^

vim +61 lib/iov_iter.c

55 n -= __v.iov_len; \
56 } \
57 n = wanted; \
58 }
59
60 #define iterate_bvec(i, n, __v, __p, skip, STEP) { \
> 61 struct bvec_iter __bi, __start; \
62 __start.bi_size = n; \
63 __start.bi_bvec_done = skip; \
64 __start.bi_idx = 0; \
65 __p = i->bvec; \
> 66 for_each_bvec(__v, __p, __bi, __start) { \
67 (void)(STEP); \
68 } \
69 if (!__bi.bi_idx) \
70 skip += __v.bv_len; \
> 71 else \
72 skip = __v.bv_len; \
73 }
74
75 #define iterate_all_kinds(i, n, v, I, B, K) { \
76 size_t skip = i->iov_offset; \
77 if (unlikely(i->type & ITER_BVEC)) { \
78 const struct bio_vec *bvec; \
79 struct bio_vec v; \
> 80 iterate_bvec(i, n, v, bvec, skip, (B)) \
81 } else if (unlikely(i->type & ITER_KVEC)) { \
82 const struct kvec *kvec; \
83 struct kvec v; \

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: Binary data

Next message: Christoffer Dall: "Re: [PATCH v2 11/20] kvm: rename pfn_t to kvm_pfn_t"
Previous message: Russell King - ARM Linux: "Re: [PATCH] sched_clock: add data pointer argument to read callback"
In reply to: Ming Lei: "[PATCH 2/2] iov_iter: use bvec iterator to implement iterate_bvec()"
Next in thread: Ming Lei: "Re: [PATCH 2/2] iov_iter: use bvec iterator to implement iterate_bvec()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]