[Devel] [PATCH VZ9 2/2] fs/fuse/kio: remove	aligned-inside-packed kludge
    Alexey Kuznetsov 
    kuznet at virtuozzo.com
       
    Wed Oct 22 15:22:02 MSK 2025
    
    
  
Hello!
On Wed, Oct 22, 2025 at 7:26 PM Pavel Tikhomirov
<ptikhomirov at virtuozzo.com> wrote:
> I see that we still use __pre_aligned() and __pre_packed in other places
> but they are defined like:
...
> So my question is, do they actually do/did something? At least now they
> seem empty.
They are empty for gcc. Yes, these defines are also inherited from
user space, but they are used in protocol definitions.
So, they should stay because we share these defintions with user
space, which uses them with non-gcc, even with windows
(unfortunately).
But they are absolute no for internal kernel structures like pcs_msg.
> "pool" seems completely removed, is it intentional? Is it an unused field?
Yes. It also was copypasted from user space at day zero, but does not make sense
in kernel. At least for now.
I noticed it when searchin for bug fixed in patch 1 of the series. I
could not remove
it or change sick struct alignment before the bug is fixed.
    
    
More information about the Devel
mailing list