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

Sedat Dilek <sedat.dilek@xxxxxxxxxxxxxx> · Mon, 18 Apr 2011 12:22:30 +0200

On Mon, Apr 18, 2011 at 10:18 AM, Wu Fengguang <fengguang.wu@xxxxxxxxx> wrote:
>> Unfortunately, this "v2" patch still breaks with gcc-4.6 here:
>>
>> Â 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
>>
>> My patchset against next-20110415 looks like this:
>>
>> Â (+) OK Â writeback-dirty-throttling-v7/writeback-dirty-throttling-v7.patch
>> Â (+) OK Â writeback-dirty-throttling-post-v7/0001-writeback-i386-compile-fix.patch
>> Â (+) OK Â writeback-dirty-throttling-post-v7/0002-writeback-quick-CONFIG_BLK_DEV_THROTTLING-n-compile-.patch
>> Â (+) OK Â writeback-dirty-throttling-post-v7/0003-Revert-writeback-scale-dirty-proportions-period-with.patch
>> Â (+) OK Â writeback-dirty-throttling-v7-fix/writeback-scale-dirty-proportions-period-with-writeout-bandwidth-v2.patch
>>
>> Attached are the disasm of mm/page-writeback.o (v2, gcc-4.6) and the
>> disasm of yesterday's experiments with gcc-4.5.
>>
>> [ gcc-4.5 ]
>>
>> 00006574 <calc_period_shift>:
>> Â Â 6574: Â Â Â a1 90 00 00 00 Â Â Â Â Âmov Â Â0x90,%eax Â Â Â Â6575:
>> R_386_32 Âdefault_backing_dev_info
>> Â Â 6579: Â Â Â 55 Â Â Â Â Â Â Â Â Â Â Âpush Â %ebp
>> Â Â 657a: Â Â Â 89 e5 Â Â Â Â Â Â Â Â Â mov Â Â%esp,%ebp
>> Â Â 657c: Â Â Â e8 02 f8 ff ff Â Â Â Â Âcall Â 5d83 <__ilog2_u32>
>> Â Â 6581: Â Â Â 5d Â Â Â Â Â Â Â Â Â Â Âpop Â Â%ebp
>> Â Â 6582: Â Â Â 83 c0 02 Â Â Â Â Â Â Â Âadd Â Â$0x2,%eax
>> Â Â 6585: Â Â Â c3 Â Â Â Â Â Â Â Â Â Â Âret
>
> gcc-4.5 is generating the right code, while the below silly macro
> expansion seems like __builtin_constant_p(n) wrongly evaluating to true.
> I wonder if the attached patch will workaround the gcc's bug. It adds a
> local variable in calc_period_shift() for passing to ilog2().
>
> Thanks,
> Fengguang
>

Thanks for the patch, that "gcc-4.6 workaround" works here.
Feel free to add:

       Tested-by: Sedat Dilek <sedat.dilek@xxxxxxxxx>.

- Sedat -

P.S.: Attached disasm v3 with gcc-4.6

>> [ gcc-4.6 ]
>>
>> 000008c9 <calc_period_shift.part.10>:
>> Â Â Â8c9: Â Â Â 8b 15 90 00 00 00 Â Â Â mov Â Â0x90,%edx Â Â Â Â8cb:
>> R_386_32 Â default_backing_dev_info
>> Â Â Â8cf: Â Â Â 55 Â Â Â Â Â Â Â Â Â Â Âpush Â %ebp
>> Â Â Â8d0: Â Â Â 89 e5 Â Â Â Â Â Â Â Â Â mov Â Â%esp,%ebp
>> Â Â Â8d2: Â Â Â 85 d2 Â Â Â Â Â Â Â Â Â test Â %edx,%edx
>> Â Â Â8d4: Â Â Â 0f 88 46 01 00 00 Â Â Â js Â Â a20
>> <calc_period_shift.part.10+0x157>
>> Â Â Â8da: Â Â Â f7 c2 00 00 00 40 Â Â Â test Â $0x40000000,%edx
>> Â Â Â8e0: Â Â Â b8 20 00 00 00 Â Â Â Â Âmov Â Â$0x20,%eax
>> Â Â Â8e5: Â Â Â 0f 85 3a 01 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â8eb: Â Â Â f7 c2 00 00 00 20 Â Â Â test Â $0x20000000,%edx
>> Â Â Â8f1: Â Â Â b0 1f Â Â Â Â Â Â Â Â Â mov Â Â$0x1f,%al
>> Â Â Â8f3: Â Â Â 0f 85 2c 01 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â8f9: Â Â Â f7 c2 00 00 00 10 Â Â Â test Â $0x10000000,%edx
>> Â Â Â8ff: Â Â Â b0 1e Â Â Â Â Â Â Â Â Â mov Â Â$0x1e,%al
>> Â Â Â901: Â Â Â 0f 85 1e 01 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â907: Â Â Â f7 c2 00 00 00 08 Â Â Â test Â $0x8000000,%edx
>> Â Â Â90d: Â Â Â b0 1d Â Â Â Â Â Â Â Â Â mov Â Â$0x1d,%al
>> Â Â Â90f: Â Â Â 0f 85 10 01 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â915: Â Â Â f7 c2 00 00 00 04 Â Â Â test Â $0x4000000,%edx
>> Â Â Â91b: Â Â Â b0 1c Â Â Â Â Â Â Â Â Â mov Â Â$0x1c,%al
>> Â Â Â91d: Â Â Â 0f 85 02 01 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â923: Â Â Â f7 c2 00 00 00 02 Â Â Â test Â $0x2000000,%edx
>> Â Â Â929: Â Â Â b0 1b Â Â Â Â Â Â Â Â Â mov Â Â$0x1b,%al
>> Â Â Â92b: Â Â Â 0f 85 f4 00 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â931: Â Â Â f7 c2 00 00 00 01 Â Â Â test Â $0x1000000,%edx
>> Â Â Â937: Â Â Â b0 1a Â Â Â Â Â Â Â Â Â mov Â Â$0x1a,%al
>> Â Â Â939: Â Â Â 0f 85 e6 00 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â93f: Â Â Â f7 c2 00 00 80 00 Â Â Â test Â $0x800000,%edx
>> Â Â Â945: Â Â Â b0 19 Â Â Â Â Â Â Â Â Â mov Â Â$0x19,%al
>> Â Â Â947: Â Â Â 0f 85 d8 00 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â94d: Â Â Â f7 c2 00 00 40 00 Â Â Â test Â $0x400000,%edx
>> Â Â Â953: Â Â Â b0 18 Â Â Â Â Â Â Â Â Â mov Â Â$0x18,%al
>> Â Â Â955: Â Â Â 0f 85 ca 00 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â95b: Â Â Â f7 c2 00 00 20 00 Â Â Â test Â $0x200000,%edx
>> Â Â Â961: Â Â Â b0 17 Â Â Â Â Â Â Â Â Â mov Â Â$0x17,%al
>> Â Â Â963: Â Â Â 0f 85 bc 00 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â969: Â Â Â f7 c2 00 00 10 00 Â Â Â test Â $0x100000,%edx
>> Â Â Â96f: Â Â Â b0 16 Â Â Â Â Â Â Â Â Â mov Â Â$0x16,%al
>> Â Â Â971: Â Â Â 0f 85 ae 00 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â977: Â Â Â f7 c2 00 00 08 00 Â Â Â test Â $0x80000,%edx
>> Â Â Â97d: Â Â Â b0 15 Â Â Â Â Â Â Â Â Â mov Â Â$0x15,%al
>> Â Â Â97f: Â Â Â 0f 85 a0 00 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â985: Â Â Â f7 c2 00 00 04 00 Â Â Â test Â $0x40000,%edx
>> Â Â Â98b: Â Â Â b0 14 Â Â Â Â Â Â Â Â Â mov Â Â$0x14,%al
>> Â Â Â98d: Â Â Â 0f 85 92 00 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â993: Â Â Â f7 c2 00 00 02 00 Â Â Â test Â $0x20000,%edx
>> Â Â Â999: Â Â Â b0 13 Â Â Â Â Â Â Â Â Â mov Â Â$0x13,%al
>> Â Â Â99b: Â Â Â 0f 85 84 00 00 00 Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9a1: Â Â Â f7 c2 00 00 01 00 Â Â Â test Â $0x10000,%edx
>> Â Â Â9a7: Â Â Â b0 12 Â Â Â Â Â Â Â Â Â mov Â Â$0x12,%al
>> Â Â Â9a9: Â Â Â 75 7a Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9ab: Â Â Â f6 c6 80 Â Â Â Â Â Â Â Âtest Â $0x80,%dh
>> Â Â Â9ae: Â Â Â b0 11 Â Â Â Â Â Â Â Â Â mov Â Â$0x11,%al
>> Â Â Â9b0: Â Â Â 75 73 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9b2: Â Â Â f6 c6 40 Â Â Â Â Â Â Â Âtest Â $0x40,%dh
>> Â Â Â9b5: Â Â Â b0 10 Â Â Â Â Â Â Â Â Â mov Â Â$0x10,%al
>> Â Â Â9b7: Â Â Â 75 6c Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9b9: Â Â Â f6 c6 20 Â Â Â Â Â Â Â Âtest Â $0x20,%dh
>> Â Â Â9bc: Â Â Â b0 0f Â Â Â Â Â Â Â Â Â mov Â Â$0xf,%al
>> Â Â Â9be: Â Â Â 75 65 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9c0: Â Â Â f6 c6 10 Â Â Â Â Â Â Â Âtest Â $0x10,%dh
>> Â Â Â9c3: Â Â Â b0 0e Â Â Â Â Â Â Â Â Â mov Â Â$0xe,%al
>> Â Â Â9c5: Â Â Â 75 5e Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9c7: Â Â Â f6 c6 08 Â Â Â Â Â Â Â Âtest Â $0x8,%dh
>> Â Â Â9ca: Â Â Â b0 0d Â Â Â Â Â Â Â Â Â mov Â Â$0xd,%al
>> Â Â Â9cc: Â Â Â 75 57 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9ce: Â Â Â f6 c6 04 Â Â Â Â Â Â Â Âtest Â $0x4,%dh
>> Â Â Â9d1: Â Â Â b0 0c Â Â Â Â Â Â Â Â Â mov Â Â$0xc,%al
>> Â Â Â9d3: Â Â Â 75 50 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9d5: Â Â Â f6 c6 02 Â Â Â Â Â Â Â Âtest Â $0x2,%dh
>> Â Â Â9d8: Â Â Â b0 0b Â Â Â Â Â Â Â Â Â mov Â Â$0xb,%al
>> Â Â Â9da: Â Â Â 75 49 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9dc: Â Â Â f6 c6 01 Â Â Â Â Â Â Â Âtest Â $0x1,%dh
>> Â Â Â9df: Â Â Â b0 0a Â Â Â Â Â Â Â Â Â mov Â Â$0xa,%al
>> Â Â Â9e1: Â Â Â 75 42 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9e3: Â Â Â f6 c2 80 Â Â Â Â Â Â Â Âtest Â $0x80,%dl
>> Â Â Â9e6: Â Â Â b0 09 Â Â Â Â Â Â Â Â Â mov Â Â$0x9,%al
>> Â Â Â9e8: Â Â Â 75 3b Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9ea: Â Â Â f6 c2 40 Â Â Â Â Â Â Â Âtest Â $0x40,%dl
>> Â Â Â9ed: Â Â Â b0 08 Â Â Â Â Â Â Â Â Â mov Â Â$0x8,%al
>> Â Â Â9ef: Â Â Â 75 34 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9f1: Â Â Â f6 c2 20 Â Â Â Â Â Â Â Âtest Â $0x20,%dl
>> Â Â Â9f4: Â Â Â b0 07 Â Â Â Â Â Â Â Â Â mov Â Â$0x7,%al
>> Â Â Â9f6: Â Â Â 75 2d Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9f8: Â Â Â f6 c2 10 Â Â Â Â Â Â Â Âtest Â $0x10,%dl
>> Â Â Â9fb: Â Â Â b0 06 Â Â Â Â Â Â Â Â Â mov Â Â$0x6,%al
>> Â Â Â9fd: Â Â Â 75 26 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Â9ff: Â Â Â f6 c2 08 Â Â Â Â Â Â Â Âtest Â $0x8,%dl
>> Â Â Âa02: Â Â Â b0 05 Â Â Â Â Â Â Â Â Â mov Â Â$0x5,%al
>> Â Â Âa04: Â Â Â 75 1f Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Âa06: Â Â Â f6 c2 04 Â Â Â Â Â Â Â Âtest Â $0x4,%dl
>> Â Â Âa09: Â Â Â b0 04 Â Â Â Â Â Â Â Â Â mov Â Â$0x4,%al
>> Â Â Âa0b: Â Â Â 75 18 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Âa0d: Â Â Â f6 c2 02 Â Â Â Â Â Â Â Âtest Â $0x2,%dl
>> Â Â Âa10: Â Â Â b0 03 Â Â Â Â Â Â Â Â Â mov Â Â$0x3,%al
>> Â Â Âa12: Â Â Â 75 11 Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Âa14: Â Â Â 80 e2 01 Â Â Â Â Â Â Â Âand Â Â$0x1,%dl
>> Â Â Âa17: Â Â Â b0 02 Â Â Â Â Â Â Â Â Â mov Â Â$0x2,%al
>> Â Â Âa19: Â Â Â 75 0a Â Â Â Â Â Â Â Â Â jne Â Âa25
>> <calc_period_shift.part.10+0x15c>
>> Â Â Âa1b: Â Â Â e8 fc ff ff ff Â Â Â Â Âcall Â a1c
>> <calc_period_shift.part.10+0x153> Â Âa1c: R_386_PC32 ____ilog2_NaN
>> Â Â Âa20: Â Â Â b8 21 00 00 00 Â Â Â Â Âmov Â Â$0x21,%eax
>> Â Â Âa25: Â Â Â 5d Â Â Â Â Â Â Â Â Â Â Âpop Â Â%ebp
>> Â Â Âa26: Â Â Â c3 Â Â Â Â Â Â Â Â Â Â Âret
>>
>> 00000a27 <calc_period_shift>:
>> Â Â Âa27: Â Â Â 55 Â Â Â Â Â Â Â Â Â Â Âpush Â %ebp
>> Â Â Âa28: Â Â Â 83 ca ff Â Â Â Â Â Â Â Âor Â Â $0xffffffff,%edx
>> Â Â Âa2b: Â Â Â 89 e5 Â Â Â Â Â Â Â Â Â mov Â Â%esp,%ebp
>> Â Â Âa2d: Â Â Â 0f bd 05 90 00 00 00 Â Âbsr Â Â0x90,%eax Â Â Â Âa30:
>> R_386_32 Â default_backing_dev_info
>> Â Â Âa34: Â Â Â 0f 44 c2 Â Â Â Â Â Â Â Âcmove Â%edx,%eax
>> Â Â Âa37: Â Â Â 5d Â Â Â Â Â Â Â Â Â Â Âpop Â Â%ebp
>> Â Â Âa38: Â Â Â 83 c0 02 Â Â Â Â Â Â Â Âadd Â Â$0x2,%eax
>> Â Â Âa3b: Â Â Â c3 Â Â Â Â Â Â Â Â Â Â Âret
>>
>> - EOT -
>>
>> - Sedat -
>
>
>
>
Attachment:
mm_page-writeback.o_v3.disasm.xz

Description: Binary data