Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 2 Jun 2005 01:26:15 -0700
From:      "David O'Brien" <obrien@freebsd.org>
To:        Jung-uk Kim <jkim@niksun.com>
Cc:        freebsd-ppc@freebsd.org, freebsd-alpha@freebsd.org, freebsd-amd64@freebsd.org, freebsd-ia64@freebsd.org
Subject:   Re: [CALL FOR TESTERS] ReiserFS on non-i386 hardware patch
Message-ID:  <20050602082615.GA36096@dragon.NUXI.org>
In-Reply-To: <200506011833.12686.jkim@niksun.com>
References:  <429D860A.5000608@club-internet.fr> <Pine.SOC.4.61.0506011555050.28761@tea.blinkenlights.nl> <429DFEA1.8000004@club-internet.fr> <200506011833.12686.jkim@niksun.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jun 01, 2005 at 06:33:12PM -0400, Jung-uk Kim wrote:
> The kernel module doesn't load because of an amd64-specific GCC 3.4
> optimization bug.  To work around:
> 
> Index: Makefile
> ===================================================================
> RCS file: /home/ncvs/src/sys/modules/reiserfs/Makefile,v
> retrieving revision 1.1
> diff -u -r1.1 Makefile
> --- Makefile    24 May 2005 12:30:13 -0000      1.1
> +++ Makefile    1 Jun 2005 22:22:52 -0000
> @@ -8,4 +8,8 @@
>         reiserfs_namei.c reiserfs_prints.c reiserfs_stree.c \
>         reiserfs_vfsops.c reiserfs_vnops.c
>  
> +.if ${MACHINE_ARCH} == "amd64"
> +CFLAGS+=       -minline-all-stringops
> +.endif
> +

Unfortunately we really don't like to put this type of CFLAGS in non-*.mk
files.  Since we know the GCC 3.4.2 compiler bugs will be fixed, I
suggest we just document this issue and be happy that otherwise the code
is ready to go on FreeBSD/amd64.

-- 
-- David  (obrien@FreeBSD.org)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050602082615.GA36096>