On 05-17 20:31, Mike Rapoport wrote:
> On Thu, May 14, 2026 at 10:26:23PM +0000, Pasha Tatashin wrote:
> > Currently, luo_session_setup_outgoing() allocates the session block and
> > sets its physical address in the header immediately. With upcoming
> > dynamic block-based session management, this makes the first block
> > different from the rest. Move the allocation to where it is first needed.
> >
> > Signed-off-by: Pasha Tatashin <[email protected]>
>
> Acked-by: Mike Rapoport (Microsoft) <[email protected]>
>
> > ---
> > @@ -77,15 +77,16 @@
> >
> > /**
> > * struct luo_session_header - Header struct for managing LUO sessions.
> > - * @count: The number of sessions currently tracked in the @list.
> > - * @list: The head of the linked list of `struct luo_session`
> > instances.
> > - * @rwsem: A read-write semaphore providing synchronized access to the
> > - * session list and other fields in this structure.
> > - * @header_ser: The header data of serialization array.
> > - * @ser: The serialized session data (an array of
> > - * `struct luo_session_ser`).
> > - * @active: Set to true when first initialized. If previous kernel did
> > not
> > - * send session data, active stays false for incoming.
> > + * @count: The number of sessions currently tracked in the @list.
> > + * @list: The head of the linked list of `struct luo_session`
> > instances.
> > + * @rwsem: A read-write semaphore providing synchronized access to
> > the
> > + * session list and other fields in this structure.
> > + * @header_ser: The header data of serialization array.
> > + * @ser: The serialized session data (an array of
> > + * `struct luo_session_ser`).
> > + * @sessions_pa: Points to the location of sessions_pa within struct
> > luo_ser.
> > + * @active: Set to true when first initialized. If previous kernel
> > did not
> > + * send session data, active stays false for incoming.
>
> Hmm, why addition of a single field changed the entire block? :/
Yes, I had to decide whether to shorten the field name or change the
entire block. I opted for the latter. :-)
>
> > */
> > struct luo_session_header {
> > long count;
>
> --
> Sincerely yours,
> Mike.
