linux-kernel - Re: [PATCH 00/12] IO-less dirty throttling v7

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110417041003.GA17032@localhost>
Date:	Sun, 17 Apr 2011 12:10:03 +0800
From:	Wu Fengguang <fengguang.wu@...el.com>
To:	"sedat.dilek@...il.com" <sedat.dilek@...il.com>
Cc:	Andrew Morton <akpm@...ux-foundation.org>, Jan Kara <jack@...e.cz>,
	Christoph Hellwig <hch@....de>,
	Trond Myklebust <Trond.Myklebust@...app.com>,
	Dave Chinner <david@...morbit.com>,
	Theodore Ts'o <tytso@....edu>,
	Chris Mason <chris.mason@...cle.com>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	Mel Gorman <mel@....ul.ie>, Rik van Riel <riel@...hat.com>,
	KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>,
	Greg Thelen <gthelen@...gle.com>,
	Minchan Kim <minchan.kim@...il.com>,
	Vivek Goyal <vgoyal@...hat.com>,
	Andrea Righi <arighi@...eler.com>,
	Balbir Singh <balbir@...ux.vnet.ibm.com>,
	linux-mm <linux-mm@...ck.org>,
	"linux-fsdevel@...r.kernel.org" <linux-fsdevel@...r.kernel.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 00/12] IO-less dirty throttling v7

On Sun, Apr 17, 2011 at 11:18:43AM +0800, Sedat Dilek wrote:
> On Sun, Apr 17, 2011 at 3:44 AM, Wu Fengguang <fengguang.wu@...el.com> wrote:
> > Hi Sedat,
> >
> > On Sun, Apr 17, 2011 at 12:27:58AM +0800, Sedat Dilek wrote:
> >
> >> I pulled your tree into linux-next (next-20110415) on an i386 Debian host.
> >>
> >> My build breaks here:
> >> ...
> >>   MODPOST vmlinux.o
> >>   GEN     .version
> >>   CHK     include/generated/compile.h
> >>   UPD     include/generated/compile.h
> >>   CC      init/version.o
> >>   LD      init/built-in.o
> >>   LD      .tmp_vmlinux1
> >> mm/built-in.o: In function `bdi_position_ratio':
> >> page-writeback.c:(.text+0x5c83): undefined reference to `__udivdi3'
> >
> > Yes it can be fixed by the attached patch.
> >
> >> mm/built-in.o: In function `calc_period_shift.part.10':
> >> page-writeback.c:(.text+0x6446): undefined reference to `____ilog2_NaN'
> >
> > I cannot reproduce this error. In the git tree, calc_period_shift() is
> > actually quite simple:
> >
> > static int calc_period_shift(void)
> > {
> >        return 2 + ilog2(default_backing_dev_info.avg_write_bandwidth);
> > }
> >
> > where avg_write_bandwidth is of type "unsigned long".
> >
> >> make[4]: *** [.tmp_vmlinux1] Error
> >>
> >> BTW, which kernel-config options have to be set besides
> >> CONFIG_BLK_DEV_THROTTLING=y?
> >
> > No. I used your kconfig on 2.6.39-rc3 and it compiles OK for i386.
> >
> > I've pushed two patches into the git tree fixing the compile errors.
> > Thank you for trying it out and report!
> >
> > Thanks,
> > Fengguang
> >
> 
> Thanks for your patch.
> 
> The 1st part of the build-error is gone, but 2nd part still remains:
> 
>   LD      .tmp_vmlinux1
> mm/built-in.o: In function `calc_period_shift.part.10':
> page-writeback.c:(.text+0x6458): undefined reference to `____ilog2_NaN'
> make[4]: *** [.tmp_vmlinux1] Error 1
> 
> I have attached some disasm-ed files.

OK. I tried next-20110415 and your kconfig and still got no error.

Please revert the last commit. It's not necessary anyway.

commit 84a9890ddef487d9c6d70934c0a2addc65923bcf
Author: Wu Fengguang <fengguang.wu@...el.com>
Date:   Sat Apr 16 18:38:41 2011 -0600

    writeback: scale dirty proportions period with writeout bandwidth
    
    CC: Peter Zijlstra <a.p.zijlstra@...llo.nl>
    Signed-off-by: Wu Fengguang <fengguang.wu@...el.com>

> Unfortunately, I don't see any new commits in your GIT tree.

Yeah I cannot see it in the web interface, too:

http://git.kernel.org/?p=linux/kernel/git/wfg/writeback.git;a=shortlog;h=refs/heads/dirty-throttling-v7

But they are in the dirty-throttling-v7 branch at kernel.org:

commit d0e30163e390d87387ec13e3b1c2168238c26793
Author: Wu Fengguang <fengguang.wu@...el.com>
Date:   Sun Apr 17 11:59:12 2011 +0800

    Revert "writeback: scale dirty proportions period with writeout bandwidth"
    
    This reverts commit 84a9890ddef487d9c6d70934c0a2addc65923bcf.
    
    sedat.dilek@...il.com:
    
        LD      .tmp_vmlinux1
      mm/built-in.o: In function `calc_period_shift.part.10':
      page-writeback.c:(.text+0x6458): undefined reference to `____ilog2_NaN'
      make[4]: *** [.tmp_vmlinux1] Error 1

commit fc5c8b04119a5bcc46865e66eec3e6133ecb56e9
Author: Wu Fengguang <fengguang.wu@...el.com>
Date:   Sun Apr 17 09:22:41 2011 -0600

    writeback: quick CONFIG_BLK_DEV_THROTTLING=n compile fix
    
    Reported-by: Sedat Dilek <sedat.dilek@...glemail.com>
    Signed-off-by: Wu Fengguang <fengguang.wu@...el.com>

commit c4a7e3f48dcfae71d0e3d2c55dcc381b3def1919
Author: Wu Fengguang <fengguang.wu@...el.com>
Date:   Sun Apr 17 09:04:44 2011 -0600

    writeback: i386 compile fix
    
    mm/built-in.o: In function `bdi_position_ratio':
    page-writeback.c:(.text+0x5c83): undefined reference to `__udivdi3'
    mm/built-in.o: In function `calc_period_shift.part.10':
    page-writeback.c:(.text+0x6446): undefined reference to `____ilog2_NaN'
    make[4]: *** [.tmp_vmlinux1] Error
    
    Reported-by: Sedat Dilek <sedat.dilek@...glemail.com>
    Signed-off-by: Wu Fengguang <fengguang.wu@...el.com>


Thanks,
Fengguang
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/