NetBSD-Bugs archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: toolchain/58663: gpt(1) biosboot.c doesn't compile on modern Linux
The following reply was made to PR toolchain/58663; it has been noted by GNATS.
From: Jason Thorpe <thorpej%me.com@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: toolchain-manager%netbsd.org@localhost,
gnats-admin%netbsd.org@localhost,
netbsd-bugs%netbsd.org@localhost,
Lloyd Parkes <lloyd%must-have-coffee.gen.nz@localhost>
Subject: Re: toolchain/58663: gpt(1) biosboot.c doesn't compile on modern
Linux
Date: Wed, 4 Sep 2024 06:31:47 -0700
> On Sep 3, 2024, at 9:35=E2=80=AFPM, matthew green via gnats =
<gnats-admin%NetBSD.org@localhost> wrote:
>=20
> The following reply was made to PR toolchain/58663; it has been noted =
by GNATS.
>=20
> From: matthew green <mrg%eterna23.net@localhost>
> To: gnats-bugs%netbsd.org@localhost
> Cc: toolchain-manager%netbsd.org@localhost, gnats-admin%netbsd.org@localhost,
> netbsd-bugs%netbsd.org@localhost
> Subject: re: toolchain/58663: gpt(1) biosboot.c doesn't compile on =
modern Linux
> Date: Wed, 04 Sep 2024 14:33:04 +1000
>=20
>> - daddr_t start =3D 0;
>> + off_t start =3D 0; /* off_t because of gpt_human_get() =
*/
>=20
> i think your change is generally correct, but it really indicates
> it much larger problem.
>=20
> daddr_t should be made to be 64-bit. (except for the cases where =
netbsd
> is building 32-bit daddr_t values for eg, old platform boot programs.)
Be careful .. it certainly used to be the case that daddr_t was used in =
on-disk structure definitions in e.g. UFS.
-- thorpej
Home |
Main Index |
Thread Index |
Old Index