From owner-p4-projects@FreeBSD.ORG Thu Apr 24 17:44:03 2003 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 3704D37B404; Thu, 24 Apr 2003 17:44:03 -0700 (PDT) Delivered-To: perforce@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id DC18337B401 for ; Thu, 24 Apr 2003 17:44:02 -0700 (PDT) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 86E1143FB1 for ; Thu, 24 Apr 2003 17:44:02 -0700 (PDT) (envelope-from peter@freebsd.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.12.6/8.12.6) with ESMTP id h3P0i20U011308 for ; Thu, 24 Apr 2003 17:44:02 -0700 (PDT) (envelope-from peter@freebsd.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.12.6/8.12.6/Submit) id h3P0i2aA011300 for perforce@freebsd.org; Thu, 24 Apr 2003 17:44:02 -0700 (PDT) Date: Thu, 24 Apr 2003 17:44:02 -0700 (PDT) Message-Id: <200304250044.h3P0i2aA011300@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to peter@freebsd.org using -f From: Peter Wemm To: Perforce Change Reviews Subject: PERFORCE change 29658 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Apr 2003 00:44:04 -0000 http://perforce.freebsd.org/chv.cgi?CH=29658 Change 29658 by peter@peter_daintree on 2003/04/24 17:43:07 gah! gag me with a spoon! We have to deliberately odd-align the starting stack pointer because gcc "knows" that there is a return address pushed there and will push the wrong number of longs onto the stack, causing everything to be misaligned. Affected files ... .. //depot/projects/hammer/sys/x86_64/x86_64/machdep.c#80 edit Differences ... ==== //depot/projects/hammer/sys/x86_64/x86_64/machdep.c#80 (text+ko) ==== @@ -466,7 +466,8 @@ bzero((char *)regs, sizeof(struct trapframe)); regs->tf_rip = entry; - regs->tf_rsp = stack; + /* This strangeness is to ensure alignment after the implied return address */ + regs->tf_rsp = ((stack - 8) & ~0xF) + 8; regs->tf_rdi = stack; /* argv */ regs->tf_rflags = PSL_USER | (regs->tf_rflags & PSL_T); regs->tf_ss = _udatasel;