Re: [PATCH] maint: increase stack frame size limit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 05/08/2015 12:23 AM, Roman Bogorodskiy wrote:
>   Roman Bogorodskiy wrote:
> 
>>   Roman Bogorodskiy wrote:
>>
>>> Currently the stack frame size limit is set to 4096 and it started to
>>> cause an error like this:
>>>
>>> virshtest.c:253:1: error: stack frame size of 5512 bytes in function
>>> 'mymain' [-Werror,-Wframe-larger-than=]
>>> mymain(void)
>>> ^
>>> 1 error generated.
>>>
>>> Increase it to 6144.
>>
>> Worth to mention that it's started to show up after updating to clang
>> 3.6.
> 
> Any comments on this?

Have you reported a possible clang regression bug to the clang folks?
Or is the new clang version the first one to support what was originally
only gcc's -Wframe-larger-than?  At any rate, why does clang require a
larger stack than gcc?  It sounds like in addition to fixing the
virshtest.c file to quit stack-allocating such a large amount of space,
that you should also be letting the clang compiler folks know about
their weakness.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]