public inbox for linux-next@vger.kernel.org 
 help / color / mirror / Atom feed
* linux-next: build failure after merge of the final tree (nfsd tree related)
@ 2013-04-03  6:42 Stephen Rothwell
  2013-04-03 11:10 ` Jeff Layton
  0 siblings, 1 reply; 6+ messages in thread
From: Stephen Rothwell @ 2013-04-03  6:42 UTC (permalink / raw)
  To: J. Bruce Fields; +Cc: linux-next, linux-kernel, Jeff Layton

[-- Attachment #1: Type: text/plain, Size: 551 bytes --]

Hi all,

After merging the final tree, today's linux-next build (arm defconfig)
failed like this:

fs/built-in.o: In function `nfsd_reply_cache_stats_show':
super.c:(.text+0x87308): undefined reference to `__udivdi3'

Probably caused by commit 187da2f90879 ("nfsd: keep track of the max and
average time to search the cache") which adds a divide by u64
(num_searches).

I have reverted that commit and the following one for today.
-- 
Cheers,
Stephen Rothwell                    sfr@canb•auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux-next: build failure after merge of the final tree (nfsd tree related)
  2013-04-03  6:42 linux-next: build failure after merge of the final tree (nfsd tree related) Stephen Rothwell
@ 2013-04-03 11:10 ` Jeff Layton
  2013-04-03 11:33   ` J. Bruce Fields
  0 siblings, 1 reply; 6+ messages in thread
From: Jeff Layton @ 2013-04-03 11:10 UTC (permalink / raw)
  To: Stephen Rothwell; +Cc: J. Bruce Fields, linux-next, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 767 bytes --]

On Wed, 3 Apr 2013 17:42:19 +1100
Stephen Rothwell <sfr@canb•auug.org.au> wrote:

> Hi all,
> 
> After merging the final tree, today's linux-next build (arm defconfig)
> failed like this:
> 
> fs/built-in.o: In function `nfsd_reply_cache_stats_show':
> super.c:(.text+0x87308): undefined reference to `__udivdi3'
> 
> Probably caused by commit 187da2f90879 ("nfsd: keep track of the max and
> average time to search the cache") which adds a divide by u64
> (num_searches).
> 
> I have reverted that commit and the following one for today.

Thanks, known problem...

Looks like Bruce's tree has an older version of that patch series. I
think we just need to get him to drop that one and merge the new one.

-- 
Jeff Layton <jlayton@redhat•com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux-next: build failure after merge of the final tree (nfsd tree related)
  2013-04-03 11:10 ` Jeff Layton
@ 2013-04-03 11:33   ` J. Bruce Fields
  2013-04-03 11:38     ` Jeff Layton
  0 siblings, 1 reply; 6+ messages in thread
From: J. Bruce Fields @ 2013-04-03 11:33 UTC (permalink / raw)
  To: Jeff Layton; +Cc: Stephen Rothwell, linux-next, linux-kernel

On Wed, Apr 03, 2013 at 07:10:54AM -0400, Jeff Layton wrote:
> On Wed, 3 Apr 2013 17:42:19 +1100
> Stephen Rothwell <sfr@canb•auug.org.au> wrote:
> 
> > Hi all,
> > 
> > After merging the final tree, today's linux-next build (arm defconfig)
> > failed like this:
> > 
> > fs/built-in.o: In function `nfsd_reply_cache_stats_show':
> > super.c:(.text+0x87308): undefined reference to `__udivdi3'
> > 
> > Probably caused by commit 187da2f90879 ("nfsd: keep track of the max and
> > average time to search the cache") which adds a divide by u64
> > (num_searches).
> > 
> > I have reverted that commit and the following one for today.
> 
> Thanks, known problem...
> 
> Looks like Bruce's tree has an older version of that patch series. I
> think we just need to get him to drop that one and merge the new one.

Arrgh, sorry--could you remind me which is the new one?

--b.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux-next: build failure after merge of the final tree (nfsd tree related)
  2013-04-03 11:33   ` J. Bruce Fields
@ 2013-04-03 11:38     ` Jeff Layton
  2013-04-03 18:05       ` J. Bruce Fields
  0 siblings, 1 reply; 6+ messages in thread
From: Jeff Layton @ 2013-04-03 11:38 UTC (permalink / raw)
  To: J. Bruce Fields; +Cc: Stephen Rothwell, linux-next, linux-kernel

On Wed, 3 Apr 2013 07:33:01 -0400
"J. Bruce Fields" <bfields@fieldses•org> wrote:

> On Wed, Apr 03, 2013 at 07:10:54AM -0400, Jeff Layton wrote:
> > On Wed, 3 Apr 2013 17:42:19 +1100
> > Stephen Rothwell <sfr@canb•auug.org.au> wrote:
> > 
> > > Hi all,
> > > 
> > > After merging the final tree, today's linux-next build (arm defconfig)
> > > failed like this:
> > > 
> > > fs/built-in.o: In function `nfsd_reply_cache_stats_show':
> > > super.c:(.text+0x87308): undefined reference to `__udivdi3'
> > > 
> > > Probably caused by commit 187da2f90879 ("nfsd: keep track of the max and
> > > average time to search the cache") which adds a divide by u64
> > > (num_searches).
> > > 
> > > I have reverted that commit and the following one for today.
> > 
> > Thanks, known problem...
> > 
> > Looks like Bruce's tree has an older version of that patch series. I
> > think we just need to get him to drop that one and merge the new one.
> 
> Arrgh, sorry--could you remind me which is the new one?
> 

It was the one I sent on 3/19. Those patches (plus a couple more) are
also in the current nfsd-3.10 branch of my git tree too, so it may be
easiest to just pick them from there.

Thanks,
-- 
Jeff Layton <jlayton@redhat•com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux-next: build failure after merge of the final tree (nfsd tree related)
  2013-04-03 11:38     ` Jeff Layton
@ 2013-04-03 18:05       ` J. Bruce Fields
  2013-04-03 18:16         ` Jeff Layton
  0 siblings, 1 reply; 6+ messages in thread
From: J. Bruce Fields @ 2013-04-03 18:05 UTC (permalink / raw)
  To: Jeff Layton; +Cc: Stephen Rothwell, linux-next, linux-kernel

On Wed, Apr 03, 2013 at 07:38:57AM -0400, Jeff Layton wrote:
> On Wed, 3 Apr 2013 07:33:01 -0400
> "J. Bruce Fields" <bfields@fieldses•org> wrote:
> 
> > On Wed, Apr 03, 2013 at 07:10:54AM -0400, Jeff Layton wrote:
> > > On Wed, 3 Apr 2013 17:42:19 +1100
> > > Stephen Rothwell <sfr@canb•auug.org.au> wrote:
> > > 
> > > > Hi all,
> > > > 
> > > > After merging the final tree, today's linux-next build (arm defconfig)
> > > > failed like this:
> > > > 
> > > > fs/built-in.o: In function `nfsd_reply_cache_stats_show':
> > > > super.c:(.text+0x87308): undefined reference to `__udivdi3'
> > > > 
> > > > Probably caused by commit 187da2f90879 ("nfsd: keep track of the max and
> > > > average time to search the cache") which adds a divide by u64
> > > > (num_searches).
> > > > 
> > > > I have reverted that commit and the following one for today.
> > > 
> > > Thanks, known problem...
> > > 
> > > Looks like Bruce's tree has an older version of that patch series. I
> > > think we just need to get him to drop that one and merge the new one.
> > 
> > Arrgh, sorry--could you remind me which is the new one?
> > 
> 
> It was the one I sent on 3/19. Those patches (plus a couple more) are
> also in the current nfsd-3.10 branch of my git tree too, so it may be
> easiest to just pick them from there.

I hate rewriting that branch, but OK, done: does my for-3.10 look right
to you now?

(It's still missing some of your latest patches.)

--b.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux-next: build failure after merge of the final tree (nfsd tree related)
  2013-04-03 18:05       ` J. Bruce Fields
@ 2013-04-03 18:16         ` Jeff Layton
  0 siblings, 0 replies; 6+ messages in thread
From: Jeff Layton @ 2013-04-03 18:16 UTC (permalink / raw)
  To: J. Bruce Fields; +Cc: Stephen Rothwell, linux-next, linux-kernel

On Wed, 3 Apr 2013 14:05:19 -0400
"J. Bruce Fields" <bfields@fieldses•org> wrote:

> On Wed, Apr 03, 2013 at 07:38:57AM -0400, Jeff Layton wrote:
> > On Wed, 3 Apr 2013 07:33:01 -0400
> > "J. Bruce Fields" <bfields@fieldses•org> wrote:
> > 
> > > On Wed, Apr 03, 2013 at 07:10:54AM -0400, Jeff Layton wrote:
> > > > On Wed, 3 Apr 2013 17:42:19 +1100
> > > > Stephen Rothwell <sfr@canb•auug.org.au> wrote:
> > > > 
> > > > > Hi all,
> > > > > 
> > > > > After merging the final tree, today's linux-next build (arm defconfig)
> > > > > failed like this:
> > > > > 
> > > > > fs/built-in.o: In function `nfsd_reply_cache_stats_show':
> > > > > super.c:(.text+0x87308): undefined reference to `__udivdi3'
> > > > > 
> > > > > Probably caused by commit 187da2f90879 ("nfsd: keep track of the max and
> > > > > average time to search the cache") which adds a divide by u64
> > > > > (num_searches).
> > > > > 
> > > > > I have reverted that commit and the following one for today.
> > > > 
> > > > Thanks, known problem...
> > > > 
> > > > Looks like Bruce's tree has an older version of that patch series. I
> > > > think we just need to get him to drop that one and merge the new one.
> > > 
> > > Arrgh, sorry--could you remind me which is the new one?
> > > 
> > 
> > It was the one I sent on 3/19. Those patches (plus a couple more) are
> > also in the current nfsd-3.10 branch of my git tree too, so it may be
> > easiest to just pick them from there.
> 
> I hate rewriting that branch, but OK, done: does my for-3.10 look right
> to you now?
> 
> (It's still missing some of your latest patches.)
> 
> --b.

Yeah, sorry for that... I didn't find the problem with __udivdi3
until after I had asked you to merge the earlier set. Mea culpa...

Latest branch looks good. It would be good to get the later patches in
too, but those are less important than the DRC ones.

Thanks,
-- 
Jeff Layton <jlayton@redhat•com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2013-04-03 18:16 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-03  6:42 linux-next: build failure after merge of the final tree (nfsd tree related) Stephen Rothwell
2013-04-03 11:10 ` Jeff Layton
2013-04-03 11:33   ` J. Bruce Fields
2013-04-03 11:38     ` Jeff Layton
2013-04-03 18:05       ` J. Bruce Fields
2013-04-03 18:16         ` Jeff Layton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox