From owner-freebsd-arch Tue Apr 2 21:21:41 2002 Delivered-To: freebsd-arch@freebsd.org Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.86.163]) by hub.freebsd.org (Postfix) with ESMTP id BF9AC37B405 for ; Tue, 2 Apr 2002 21:21:35 -0800 (PST) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.12.2/8.12.2) with ESMTP id g335L94F027396; Wed, 3 Apr 2002 07:21:09 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Marcel Moolenaar Cc: arch@FreeBSD.ORG Subject: Re: Please review: endian invariant kernel dump headers In-Reply-To: Your message of "Tue, 02 Apr 2002 19:42:08 -0800." <20020403034208.GA929@dhcp01.pn.xcllnt.net> Date: Wed, 03 Apr 2002 07:21:09 +0200 Message-ID: <27395.1017811269@critter.freebsd.dk> From: Poul-Henning Kamp Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG In message <20020403034208.GA929@dhcp01.pn.xcllnt.net>, Marcel Moolenaar writes : >Please review the attached patch. The change achieves the following: > >1. Dump the kernel header in dump byte order. This is the same > as network byte order. Parity calculation is endianness > invariant and should not use the macros. good. >2. The kernel dump header had a size of 520 bytes on 64-bit > architectures due to alignment of the uint64_t following > the uint32_t. Reordering solves that. My fault, I thought I had that right. >3. No version bump is required, because all existing headers are > in little-endian and thus will not have the same version as > the big-endian dumps. I did not add support in savecore to > read version 0x01000000 headers :-) Cool. Suggest you add: CTASSERT(sizeof kerneldumpheader == 512); while at it. >If there are no blocking objections I like to commit this quickly >due to point 2. By all means. -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message