Re: [PATCH v2] x86/boot: save fields explicitly, zero out everything else

2019-08-10 Thread Chris Clayton
On 31/07/2019 06:46, john.hubb...@gmail.com wrote: > From: John Hubbard > > Recent gcc compilers (gcc 9.1) generate warnings about an > out of bounds memset, if you trying memset across several fields > of a struct. This generated a couple of warnings on x86_64 builds. > > Fix this by

Re: [PATCH v2] x86/boot: save fields explicitly, zero out everything else

2019-08-07 Thread John Hubbard
On 8/7/19 4:41 AM, David Laight wrote: > From: john.hubb...@gmail.com >> Sent: 31 July 2019 06:46 ... >> if (boot_params->sentinel) { >> -/* fields in boot_params are left uninitialized, clear them */ >> -boot_params->acpi_rsdp_addr = 0; >> -

RE: [PATCH v2] x86/boot: save fields explicitly, zero out everything else

2019-08-07 Thread David Laight
From: john.hubb...@gmail.com > Sent: 31 July 2019 06:46 > > Recent gcc compilers (gcc 9.1) generate warnings about an > out of bounds memset, if you trying memset across several fields > of a struct. This generated a couple of warnings on x86_64 builds. > > Fix this by explicitly saving the

[PATCH v2] x86/boot: save fields explicitly, zero out everything else

2019-07-30 Thread john . hubbard
From: John Hubbard Recent gcc compilers (gcc 9.1) generate warnings about an out of bounds memset, if you trying memset across several fields of a struct. This generated a couple of warnings on x86_64 builds. Fix this by explicitly saving the fields in struct boot_params that are intended to be