lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Mon, 27 Apr 2020 13:20:09 -0700 (PDT)
From:   David Miller <davem@...emloft.net>
To:     irusskikh@...vell.com
Cc:     kuba@...nel.org, netdev@...r.kernel.org, mstarovoitov@...vell.com,
        dbogdanov@...vell.com
Subject: Re: [EXT] Re: [PATCH net-next 08/17] net: atlantic: A2
 driver-firmware interface

From: Igor Russkikh <irusskikh@...vell.com>
Date: Mon, 27 Apr 2020 23:04:24 +0300

> This means I have to dig each and every structure in this header and
> understand whether it may suffer from implicit alignment/holes or not.
> 
> Not mentioning the fact that these alignment rules are different on other
> compilers, or on say 32-bit archs.
> 
> I also see a lot of code through the kernel using pack(1) for the exact same
> reason - declare hw sensitive structures and eliminate any unexpected holes.

Your resistence to this feedback is becomming irritating.

Just because something is used elsewhere doesn't mean you are open to
do the same, there is a lot of code where issues like this have not
been caught through reivew and the code still ended up in the tree.

Using packed arbitrarily is being lazy and will result in suboptimal
code generation on several platforms.

Fixed sized types have well defined padding on _all_ cpus and targets,
so if you use them properly and pad up your structures, there is
absolutely _nothing_ to worry about.

When I was very active writing hardware drivers with many HW defined
structures and whatnot, I never once considered packed.  It never even
crossed my mind, because I simply defined the data structure properly
with well defined fixed sized types and padded them out as necessary.

So please stop pushing back on this feedback and get rid of the packed
attribute.

Thank you.

Powered by blists - more mailing lists