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? :/ > */ > struct luo_session_header { > long count; -- Sincerely yours, Mike.
