| Message ID | 20230821225145.2169848-1-mjguzik@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | maple_tree: shrink struct maple_tree from 24 to 16 bytes on LP64 | expand |
On Tue, Aug 22, 2023 at 12:51:45AM +0200, Mateusz Guzik wrote: > by plugging a padding hole. > > [sounds like an obvious thing to do, am I missing something?] That was how I originally had it. Not sure when/why it got messed up.
* Mateusz Guzik <mjguzik@gmail.com> [230821 18:52]: > by plugging a padding hole. > > [sounds like an obvious thing to do, am I missing something?] Thanks for the patch. There is no reason for the current ordering. I've tested both 32 and 64 bit userspace testing to double check everything still works. > > Signed-off-by: Mateusz Guzik <mjguzik@gmail.com> Reviewed-by: Liam R. Howlett <Liam.Howlett@oracle.com> > --- > include/linux/maple_tree.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/maple_tree.h b/include/linux/maple_tree.h > index 295548cca8b3..5bdd3ce34cb9 100644 > --- a/include/linux/maple_tree.h > +++ b/include/linux/maple_tree.h > @@ -212,8 +212,8 @@ struct maple_tree { > spinlock_t ma_lock; > lockdep_map_p ma_external_lock; > }; > - void __rcu *ma_root; > unsigned int ma_flags; > + void __rcu *ma_root; > }; > > /** > -- > 2.39.2 > >
diff --git a/include/linux/maple_tree.h b/include/linux/maple_tree.h index 295548cca8b3..5bdd3ce34cb9 100644 --- a/include/linux/maple_tree.h +++ b/include/linux/maple_tree.h @@ -212,8 +212,8 @@ struct maple_tree { spinlock_t ma_lock; lockdep_map_p ma_external_lock; }; - void __rcu *ma_root; unsigned int ma_flags; + void __rcu *ma_root; }; /**
by plugging a padding hole. [sounds like an obvious thing to do, am I missing something?] Signed-off-by: Mateusz Guzik <mjguzik@gmail.com> --- include/linux/maple_tree.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)