* linux-next: build failure after merge of the ext4 tree
@ 2015-07-23 0:56 Stephen Rothwell
2015-07-23 16:49 ` Theodore Ts'o
2015-07-23 18:27 ` [PATCH block/for-linus] block: export bio_associate_*() and wbc_account_io() Tejun Heo
0 siblings, 2 replies; 7+ messages in thread
From: Stephen Rothwell @ 2015-07-23 0:56 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: linux-next, linux-kernel, Tejun Heo
Hi Theodore,
After merging the ext4 tree, today's linux-next build (x86_64
allmodconfig) failed like this:
ERROR: "wbc_account_io" [fs/ext4/ext4.ko] undefined!
ERROR: "bio_associate_blkcg" [fs/ext4/ext4.ko] undefined!
Caused by commit
001e4a8775f6 ("ext4: implement cgroup writeback support")
I have used the ext4 tree from next-20150722 for today.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build failure after merge of the ext4 tree
2015-07-23 0:56 linux-next: build failure after merge of the ext4 tree Stephen Rothwell
@ 2015-07-23 16:49 ` Theodore Ts'o
2015-07-23 17:23 ` Theodore Ts'o
2015-07-23 18:27 ` [PATCH block/for-linus] block: export bio_associate_*() and wbc_account_io() Tejun Heo
1 sibling, 1 reply; 7+ messages in thread
From: Theodore Ts'o @ 2015-07-23 16:49 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-next, linux-kernel, Tejun Heo
On Thu, Jul 23, 2015 at 10:56:23AM +1000, Stephen Rothwell wrote:
> Hi Theodore,
>
> After merging the ext4 tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> ERROR: "wbc_account_io" [fs/ext4/ext4.ko] undefined!
> ERROR: "bio_associate_blkcg" [fs/ext4/ext4.ko] undefined!
>
> Caused by commit
>
> 001e4a8775f6 ("ext4: implement cgroup writeback support")
>
> I have used the ext4 tree from next-20150722 for today.
Ah, I see, we're missing the include files which define the inline
dummy functions. I'll fix this in my tree, thanks.
- Ted
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build failure after merge of the ext4 tree
2015-07-23 16:49 ` Theodore Ts'o
@ 2015-07-23 17:23 ` Theodore Ts'o
2015-07-23 17:41 ` Tejun Heo
0 siblings, 1 reply; 7+ messages in thread
From: Theodore Ts'o @ 2015-07-23 17:23 UTC (permalink / raw)
To: Stephen Rothwell, linux-next, linux-kernel, Tejun Heo
On Thu, Jul 23, 2015 at 12:49:53PM -0400, Theodore Ts'o wrote:
> > After merging the ext4 tree, today's linux-next build (x86_64
> > allmodconfig) failed like this:
> >
> > ERROR: "wbc_account_io" [fs/ext4/ext4.ko] undefined!
> > ERROR: "bio_associate_blkcg" [fs/ext4/ext4.ko] undefined!
> >
> > Caused by commit
> >
> > 001e4a8775f6 ("ext4: implement cgroup writeback support")
> >
> > I have used the ext4 tree from next-20150722 for today.
>
> Ah, I see, we're missing the include files which define the inline
> dummy functions. I'll fix this in my tree, thanks.
Whoops, correction, the problem is that wbc_account_io() and
bio_asociate_blkcg() need to be exported as symbols.
Tejun, how quickly get you get a fix into linux-next? Or should I
just drop that patch for now?
- Ted
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: linux-next: build failure after merge of the ext4 tree
2015-07-23 17:23 ` Theodore Ts'o
@ 2015-07-23 17:41 ` Tejun Heo
0 siblings, 0 replies; 7+ messages in thread
From: Tejun Heo @ 2015-07-23 17:41 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: Stephen Rothwell, linux-next, linux-kernel
On Thu, Jul 23, 2015 at 01:23:43PM -0400, Theodore Ts'o wrote:
> On Thu, Jul 23, 2015 at 12:49:53PM -0400, Theodore Ts'o wrote:
> > > After merging the ext4 tree, today's linux-next build (x86_64
> > > allmodconfig) failed like this:
> > >
> > > ERROR: "wbc_account_io" [fs/ext4/ext4.ko] undefined!
> > > ERROR: "bio_associate_blkcg" [fs/ext4/ext4.ko] undefined!
> > >
> > > Caused by commit
> > >
> > > 001e4a8775f6 ("ext4: implement cgroup writeback support")
> > >
> > > I have used the ext4 tree from next-20150722 for today.
> >
> > Ah, I see, we're missing the include files which define the inline
> > dummy functions. I'll fix this in my tree, thanks.
>
> Whoops, correction, the problem is that wbc_account_io() and
> bio_asociate_blkcg() need to be exported as symbols.
>
> Tejun, how quickly get you get a fix into linux-next? Or should I
> just drop that patch for now?
I'll send the patch to add EXPORT to Jens right away.
Thanks.
--
tejun
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH block/for-linus] block: export bio_associate_*() and wbc_account_io()
2015-07-23 0:56 linux-next: build failure after merge of the ext4 tree Stephen Rothwell
2015-07-23 16:49 ` Theodore Ts'o
@ 2015-07-23 18:27 ` Tejun Heo
2015-07-23 18:30 ` Chris Mason
2015-07-23 19:36 ` Jens Axboe
1 sibling, 2 replies; 7+ messages in thread
From: Tejun Heo @ 2015-07-23 18:27 UTC (permalink / raw)
To: Jens Axboe
Cc: Theodore Ts'o, linux-next, linux-kernel, Stephen Rothwell,
Chris Mason, kernel-team
bio_associate_blkcg(), bio_associate_current() and wbc_account_io()
are used to implement cgroup writeback support for filesystems and
thus need to be exported. Export them.
Signed-off-by: Tejun Heo <tj@kernel.org>
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
Hello, Jens.
While this change isn't strictly necessary for 4.2, I think it'd
better to push it through for-linus so that it's there before any
filesystem specific changes are merged.
Thanks.
block/bio.c | 2 ++
fs/fs-writeback.c | 1 +
2 files changed, 3 insertions(+)
--- a/block/bio.c
+++ b/block/bio.c
@@ -2009,6 +2009,7 @@ int bio_associate_blkcg(struct bio *bio,
bio->bi_css = blkcg_css;
return 0;
}
+EXPORT_SYMBOL_GPL(bio_associate_blkcg);
/**
* bio_associate_current - associate a bio with %current
@@ -2039,6 +2040,7 @@ int bio_associate_current(struct bio *bi
bio->bi_css = task_get_css(current, blkio_cgrp_id);
return 0;
}
+EXPORT_SYMBOL_GPL(bio_associate_current);
/**
* bio_disassociate_task - undo bio_associate_current()
--- a/fs/fs-writeback.c
+++ b/fs/fs-writeback.c
@@ -702,6 +702,7 @@ void wbc_account_io(struct writeback_con
else
wbc->wb_tcand_bytes -= min(bytes, wbc->wb_tcand_bytes);
}
+EXPORT_SYMBOL_GPL(wbc_account_io);
/**
* inode_congested - test whether an inode is congested
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH block/for-linus] block: export bio_associate_*() and wbc_account_io()
2015-07-23 18:27 ` [PATCH block/for-linus] block: export bio_associate_*() and wbc_account_io() Tejun Heo
@ 2015-07-23 18:30 ` Chris Mason
2015-07-23 19:36 ` Jens Axboe
1 sibling, 0 replies; 7+ messages in thread
From: Chris Mason @ 2015-07-23 18:30 UTC (permalink / raw)
To: Tejun Heo
Cc: Jens Axboe, Theodore Ts'o, linux-next, linux-kernel,
Stephen Rothwell, kernel-team
On Thu, Jul 23, 2015 at 02:27:09PM -0400, Tejun Heo wrote:
> bio_associate_blkcg(), bio_associate_current() and wbc_account_io()
> are used to implement cgroup writeback support for filesystems and
> thus need to be exported. Export them.
Thanks, these were the only three I needed on the btrfs side.
-chris
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH block/for-linus] block: export bio_associate_*() and wbc_account_io()
2015-07-23 18:27 ` [PATCH block/for-linus] block: export bio_associate_*() and wbc_account_io() Tejun Heo
2015-07-23 18:30 ` Chris Mason
@ 2015-07-23 19:36 ` Jens Axboe
1 sibling, 0 replies; 7+ messages in thread
From: Jens Axboe @ 2015-07-23 19:36 UTC (permalink / raw)
To: Tejun Heo
Cc: Theodore Ts'o, linux-next, linux-kernel, Stephen Rothwell,
Chris Mason, kernel-team
On 07/23/2015 12:27 PM, Tejun Heo wrote:
> bio_associate_blkcg(), bio_associate_current() and wbc_account_io()
> are used to implement cgroup writeback support for filesystems and
> thus need to be exported. Export them.
>
> Signed-off-by: Tejun Heo <tj@kernel.org>
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
> Hello, Jens.
>
> While this change isn't strictly necessary for 4.2, I think it'd
> better to push it through for-linus so that it's there before any
> filesystem specific changes are merged.
Might as well shove it it, as it'll make the lives of others easier.
--
Jens Axboe
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-07-23 19:36 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-23 0:56 linux-next: build failure after merge of the ext4 tree Stephen Rothwell
2015-07-23 16:49 ` Theodore Ts'o
2015-07-23 17:23 ` Theodore Ts'o
2015-07-23 17:41 ` Tejun Heo
2015-07-23 18:27 ` [PATCH block/for-linus] block: export bio_associate_*() and wbc_account_io() Tejun Heo
2015-07-23 18:30 ` Chris Mason
2015-07-23 19:36 ` Jens Axboe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).