Skip to content

Commit ef98ffd

Browse files
matttbeintel-lab-lkp
authored andcommitted
mptcp: sched: reduce size for unused data
Thanks for the previous commit ("mptcp: sched: split get_subflow interface into two"), the mptcp_sched_data structure is now currently unused. This structure has been added to allow future extensions that are not ready yet. At the end, this structure will not even be used at all when mptcp_subflow bpf_iter will be supported [1]. Here is a first step to save 64 bytes on the stack for each scheduling operation. The structure is not removed yet not to break the WIP work on these extensions, but will be done when [1] will be ready and applied. Link: https://lore.kernel.org/[email protected] [1] Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
1 parent 98f1e71 commit ef98ffd

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

net/mptcp/sched.c

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ static void mptcp_sched_data_set_contexts(const struct mptcp_sock *msk,
177177
int mptcp_sched_get_send(struct mptcp_sock *msk)
178178
{
179179
struct mptcp_subflow_context *subflow;
180-
struct mptcp_sched_data data;
180+
struct mptcp_sched_data *data = NULL;
181181

182182
msk_owned_by_me(msk);
183183

@@ -198,15 +198,15 @@ int mptcp_sched_get_send(struct mptcp_sock *msk)
198198
}
199199

200200
if (msk->sched == &mptcp_sched_default || !msk->sched)
201-
return mptcp_sched_default_get_send(msk, &data);
202-
mptcp_sched_data_set_contexts(msk, &data);
203-
return msk->sched->get_send(msk, &data);
201+
return mptcp_sched_default_get_send(msk, data);
202+
mptcp_sched_data_set_contexts(msk, data);
203+
return msk->sched->get_send(msk, data);
204204
}
205205

206206
int mptcp_sched_get_retrans(struct mptcp_sock *msk)
207207
{
208208
struct mptcp_subflow_context *subflow;
209-
struct mptcp_sched_data data;
209+
struct mptcp_sched_data *data = NULL;
210210

211211
msk_owned_by_me(msk);
212212

@@ -220,10 +220,10 @@ int mptcp_sched_get_retrans(struct mptcp_sock *msk)
220220
}
221221

222222
if (msk->sched == &mptcp_sched_default || !msk->sched)
223-
return mptcp_sched_default_get_retrans(msk, &data);
223+
return mptcp_sched_default_get_retrans(msk, data);
224224

225-
mptcp_sched_data_set_contexts(msk, &data);
225+
mptcp_sched_data_set_contexts(msk, data);
226226
if (msk->sched->get_retrans)
227-
return msk->sched->get_retrans(msk, &data);
228-
return msk->sched->get_send(msk, &data);
227+
return msk->sched->get_retrans(msk, data);
228+
return msk->sched->get_send(msk, data);
229229
}

0 commit comments

Comments
 (0)