Re: [PATCH] ethtool: zero initialize coalesce struct

2019-01-18 Thread John W. Linville
On Fri, Dec 14, 2018 at 05:19:23PM -0800, Maciej Żenczykowski wrote: > From: Maciej Żenczykowski > > prior to fetching it from kernel. > > Otherwise we run the risk of very tail portion of it (dmac field) > being left entirely uninitialized, and likely containing some sort > of stale data. > >

Re: [PATCH] ethtool: zero initialize coalesce struct

2018-12-15 Thread Ben Hutchings
I handed over ethtool to John Linville some time ago. Ben. On Fri, 2018-12-14 at 17:19 -0800, Maciej Żenczykowski wrote: > From: Maciej Żenczykowski > > prior to fetching it from kernel. > > Otherwise we run the risk of very tail portion of it (dmac field) > being left entirely uninitialized,

Re: [PATCH] ethtool: zero initialize coalesce struct

2018-12-14 Thread Maciej Żenczykowski
> The dmac member of struct ethtool_coalesce was removed by commit > c0b078ce7e88 ("Revert "ethtool: Add DMA Coalescing support"") because it > has never been added in kernel (as that would break compatibility with > older ethtool versions). Hmm, well, it doesn't hurt to zero init either way...

Re: [PATCH] ethtool: zero initialize coalesce struct

2018-12-14 Thread Michal Kubecek
On Fri, Dec 14, 2018 at 05:19:23PM -0800, Maciej Żenczykowski wrote: > From: Maciej Żenczykowski > > prior to fetching it from kernel. > > Otherwise we run the risk of very tail portion of it (dmac field) > being left entirely uninitialized, and likely containing some sort > of stale data. The