Re: svn commit: r186529 - head/sys/dev/acpi_support

2009-01-01 Thread Christoph Mallon
Joerg Sonnenberger schrieb: On Sun, Dec 28, 2008 at 10:00:37PM +0100, Dimitry Andric wrote: On 2008-12-28 20:30, Stanislav Sedov wrote: - ACPI_OBJECT acpiarg[0]; + ACPI_OBJECT acpiarg[1]; I wonder how does gcc allowed this. It emits warnings only in pedantic

Re: svn commit: r186529 - head/sys/dev/acpi_support

2008-12-28 Thread Joerg Sonnenberger
On Sun, Dec 28, 2008 at 10:00:37PM +0100, Dimitry Andric wrote: > On 2008-12-28 20:30, Stanislav Sedov wrote: > >>> - ACPI_OBJECT acpiarg[0]; > >>> + ACPI_OBJECT acpiarg[1]; > > I wonder how does gcc allowed this. It emits warnings only in > > pedantic mode which we cannot u

Re: svn commit: r186529 - head/sys/dev/acpi_support

2008-12-28 Thread Dimitry Andric
On 2008-12-28 20:30, Stanislav Sedov wrote: >>> - ACPI_OBJECT acpiarg[0]; >>> + ACPI_OBJECT acpiarg[1]; > I wonder how does gcc allowed this. It emits warnings only in > pedantic mode which we cannot use to compile kernel with. Zero-sized arrays are non-standard, but ha

Re: svn commit: r186529 - head/sys/dev/acpi_support

2008-12-28 Thread Stanislav Sedov
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On Sat, 27 Dec 2008 20:50:55 -0800 Andrew Thompson mentioned: > On Sat, Dec 27, 2008 at 08:48:11PM +, Stanislav Sedov wrote: > > Author: stas > > Date: Sat Dec 27 20:48:11 2008 > > New Revision: 186529 > > URL: http://svn.freebsd.org/changeset/ba

Re: svn commit: r186529 - head/sys/dev/acpi_support

2008-12-27 Thread Andrew Thompson
On Sat, Dec 27, 2008 at 08:48:11PM +, Stanislav Sedov wrote: > Author: stas > Date: Sat Dec 27 20:48:11 2008 > New Revision: 186529 > URL: http://svn.freebsd.org/changeset/base/186529 > > Log: > - Fix incorrect array declaration that was causing the stack overflow > on some (most?) Asus

svn commit: r186529 - head/sys/dev/acpi_support

2008-12-27 Thread Stanislav Sedov
Author: stas Date: Sat Dec 27 20:48:11 2008 New Revision: 186529 URL: http://svn.freebsd.org/changeset/base/186529 Log: - Fix incorrect array declaration that was causing the stack overflow on some (most?) Asus laptops. Discussed with: rpaulo Approved by: kib (mentor) MFC aft