Re: [PATCH net-next v4 05/10] pie: rearrange structure members and their initializations
From: Gautam Ramakrishnan <hidden>
Date: 2020-01-21 15:45:04
From: Gautam Ramakrishnan <hidden>
Date: 2020-01-21 15:45:04
On Tue, Jan 21, 2020 at 8:05 PM David Miller [off-list ref] wrote:
From: gautamramk@gmail.com Date: Tue, 21 Jan 2020 19:42:44 +0530quoted
From: "Mohit P. Tahiliani" <redacted> Rearrange the members of the structures such that they appear in order of their types. Also, change the order of their initializations to match the order in which they appear in the structures. This improves the code's readability and consistency. Signed-off-by: Mohit P. Tahiliani <redacted> Signed-off-by: Leslie Monis <redacted> Signed-off-by: Gautam Ramakrishnan <redacted>What matters for structure member ordering is dense packing and grouping commonly-used-together elements for performance.
We shall reorder the variables as per their appearance in the structure and re-submit. Could you elaborate a bit on dense packing?
"order of their types" are none of those things.