Date: Tue, 02 Apr 2002 23:02:50 +0200 From: Poul-Henning Kamp <phk@critter.freebsd.dk> To: Dag-Erling Smorgrav <des@ofug.org> Cc: Robert Watson <rwatson@FreeBSD.org>, Marcel Moolenaar <marcel@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/ia64/ia64 ia64dump.c Message-ID: <34442.1017781370@critter.freebsd.dk> In-Reply-To: Your message of "02 Apr 2002 22:53:01 %2B0200." <xzpu1qtu8k2.fsf@flood.ping.uio.no>
next in thread | previous in thread | raw e-mail | index | archive | help
In message <xzpu1qtu8k2.fsf@flood.ping.uio.no>, Dag-Erling Smorgrav writes: >Poul-Henning Kamp <phk@critter.freebsd.dk> writes: >> *Currently* savecore needs nothing but the diskpartition to pull >> a kernel dump from any architecture in. > >...but it needs to know the architecture- and endianness-dependent >header format for every supported architecture, rather than just >reading and parsing a chunk of text. It is true that I have not put endianess code in savecore, but that was only due to lacking a test-case for it, certainly not for lack of the ability. The format of the header is not architecture dependent, it is the same 512 bytes of the same exact layout on all architectures. Text is nice and well for exchanging information, but it is _not_ the _only_ way to do so. Some times text is a very cumbersome way to do it, such as, in my opinion, this one. And if the ELF format Marcel has committed is indeed the defacto coredump format these days, then all you are arguing is to change the color of the ribbon around the package, and quite frankly, if you think your time is well spent on that: be my guest. But be my guest _only_ when you have commitable code, not by saying "Everybody STOP! I will do this at some unspecified time in the future". -- 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 cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?34442.1017781370>