From owner-cvs-all Mon Feb 17 23:37:42 2003 Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id F26B037B401; Mon, 17 Feb 2003 23:37:39 -0800 (PST) Received: from ns1.xcllnt.net (209-128-86-226.bayarea.net [209.128.86.226]) by mx1.FreeBSD.org (Postfix) with ESMTP id F263C43F85; Mon, 17 Feb 2003 23:37:38 -0800 (PST) (envelope-from marcel@xcllnt.net) Received: from athlon.pn.xcllnt.net (athlon.pn.xcllnt.net [192.168.4.3]) by ns1.xcllnt.net (8.12.6/8.12.6) with ESMTP id h1I7bc1o029770; Mon, 17 Feb 2003 23:37:38 -0800 (PST) (envelope-from marcel@piii.pn.xcllnt.net) Received: from athlon.pn.xcllnt.net (localhost [127.0.0.1]) by athlon.pn.xcllnt.net (8.12.7/8.12.7) with ESMTP id h1I7bc0O015313; Mon, 17 Feb 2003 23:37:38 -0800 (PST) (envelope-from marcel@athlon.pn.xcllnt.net) Received: (from marcel@localhost) by athlon.pn.xcllnt.net (8.12.7/8.12.7/Submit) id h1I7bciA015305; Mon, 17 Feb 2003 23:37:38 -0800 (PST) Date: Mon, 17 Feb 2003 23:37:38 -0800 From: Marcel Moolenaar To: Tim Robbins Cc: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/bin/sh machdep.h memalloc.c nodes.c.pat Message-ID: <20030218073738.GA3320@athlon.pn.xcllnt.net> References: <200302180208.h1I28Kbu046465@repoman.freebsd.org> <20030218022850.GB1973@athlon.pn.xcllnt.net> <20030218143222.A21049@dilbert.robbins.dropbear.id.au> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20030218143222.A21049@dilbert.robbins.dropbear.id.au> User-Agent: Mutt/1.5.3i Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Tue, Feb 18, 2003 at 02:32:22PM +1100, Tim Robbins wrote: > > Try this (ugly) patch: http://people.freebsd.org/~tjr/align.diff > I've hacked up something too. See http://www.xcllnt.net/~marcel/sh.diff Some notes: o For testing purposes I hardcoded 16-byte alignment in machdep.h. This allows me to test it on any machine I have, not just ia64. In the real world the header does not exist and is included. o I removed the stackbase completely, because you cannot depend on it being aligned properly. Hence the initial situation is with stackp NULL. o I sprinkled asserts in the code to have sh(1) fail hard at the first signs of misalignment. Give it a spin. I'm currently testing with an i386 buildworld... -- Marcel Moolenaar USPA: A-39004 marcel@xcllnt.net To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message