Skip to content

Commit

Permalink
Merge of /httpd/httpd/trunk:r1912999
Browse files Browse the repository at this point in the history
 * mod_http2: improved early cleanup of streams.



git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1913000 13f79535-47bb-0310-9956-ffa450edef68
  • Loading branch information
icing committed Oct 16, 2023
1 parent c41eb3b commit decce82
Show file tree
Hide file tree
Showing 5 changed files with 44 additions and 7 deletions.
2 changes: 2 additions & 0 deletions changes-entries/h2_cleanup.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
* mod_http2: improved early cleanup of streams.
[Stefan Eissing]
26 changes: 22 additions & 4 deletions modules/http2/h2_mplx.c
Original file line number Diff line number Diff line change
Expand Up @@ -1119,14 +1119,32 @@ static int reset_is_acceptable(h2_stream *stream)
return 1; /* otherwise, be forgiving */
}

apr_status_t h2_mplx_c1_client_rst(h2_mplx *m, int stream_id)
apr_status_t h2_mplx_c1_client_rst(h2_mplx *m, int stream_id, h2_stream *stream)
{
h2_stream *stream;
apr_status_t status = APR_SUCCESS;
int registered;

H2_MPLX_ENTER_ALWAYS(m);
stream = h2_ihash_get(m->streams, stream_id);
if (stream && !reset_is_acceptable(stream)) {
registered = (h2_ihash_get(m->streams, stream_id) != NULL);
if (!stream) {
/* a RST might arrive so late, we have already forgotten
* about it. Seems ok. */
ap_log_cerror(APLOG_MARK, APLOG_DEBUG, 0, m->c1,
H2_MPLX_MSG(m, "RST on unknown stream %d"), stream_id);
AP_DEBUG_ASSERT(!registered);
}
else if (!registered) {
/* a RST on a stream that mplx has not been told about, but
* which the session knows. Very early and annoying. */
ap_log_cerror(APLOG_MARK, APLOG_DEBUG, 0, m->c1,
H2_STRM_MSG(stream, "very early RST, drop"));
h2_stream_set_monitor(stream, NULL);
h2_stream_rst(stream, H2_ERR_STREAM_CLOSED);
h2_stream_dispatch(stream, H2_SEV_EOS_SENT);
m_stream_cleanup(m, stream);
m_be_annoyed(m);
}
else if (!reset_is_acceptable(stream)) {
m_be_annoyed(m);
}
H2_MPLX_LEAVE(m);
Expand Down
3 changes: 2 additions & 1 deletion modules/http2/h2_mplx.h
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,8 @@ int h2_mplx_c1_all_streams_want_send_data(h2_mplx *m);
* any processing going on and remove from processing
* queue.
*/
apr_status_t h2_mplx_c1_client_rst(h2_mplx *m, int stream_id);
apr_status_t h2_mplx_c1_client_rst(h2_mplx *m, int stream_id,
struct h2_stream *stream);

/**
* Get readonly access to a stream for a secondary connection.
Expand Down
18 changes: 17 additions & 1 deletion modules/http2/h2_session.c
Original file line number Diff line number Diff line change
Expand Up @@ -402,6 +402,10 @@ static int on_frame_recv_cb(nghttp2_session *ng2s,
H2_SSSN_STRM_MSG(session, frame->hd.stream_id,
"RST_STREAM by client, error=%d"),
(int)frame->rst_stream.error_code);
if (stream) {
rv = h2_stream_recv_frame(stream, NGHTTP2_RST_STREAM, frame->hd.flags,
frame->hd.length + H2_FRAME_HDR_LEN);
}
if (stream && stream->initiated_on) {
/* A stream reset on a request we sent it. Normal, when the
* client does not want it. */
Expand All @@ -410,7 +414,8 @@ static int on_frame_recv_cb(nghttp2_session *ng2s,
else {
/* A stream reset on a request it sent us. Could happen in a browser
* when the user navigates away or cancels loading - maybe. */
h2_mplx_c1_client_rst(session->mplx, frame->hd.stream_id);
h2_mplx_c1_client_rst(session->mplx, frame->hd.stream_id,
stream);
}
++session->streams_reset;
break;
Expand Down Expand Up @@ -812,6 +817,17 @@ static apr_status_t session_cleanup(h2_session *session, const char *trigger)
"goodbye, clients will be confused, should not happen"));
}

if (!h2_iq_empty(session->ready_to_process)) {
int sid;
ap_log_cerror(APLOG_MARK, APLOG_DEBUG, 0, c,
H2_SSSN_LOG(APLOGNO(), session,
"cleanup, resetting %d streams in ready-to-process"),
h2_iq_count(session->ready_to_process));
while ((sid = h2_iq_shift(session->ready_to_process)) > 0) {
h2_mplx_c1_client_rst(session->mplx, sid, get_stream(session, sid));
}
}

transit(session, trigger, H2_SESSION_ST_CLEANUP);
h2_mplx_c1_destroy(session->mplx);
session->mplx = NULL;
Expand Down
2 changes: 1 addition & 1 deletion modules/http2/h2_stream.c
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ static int trans_on_event[][H2_SS_MAX] = {
{ S_XXX, S_ERR, S_ERR, S_CL_L, S_CLS, S_XXX, S_XXX, S_XXX, },/* EV_CLOSED_L*/
{ S_ERR, S_ERR, S_ERR, S_CL_R, S_ERR, S_CLS, S_NOP, S_NOP, },/* EV_CLOSED_R*/
{ S_CLS, S_CLS, S_CLS, S_CLS, S_CLS, S_CLS, S_NOP, S_NOP, },/* EV_CANCELLED*/
{ S_NOP, S_XXX, S_XXX, S_XXX, S_XXX, S_CLS, S_CLN, S_XXX, },/* EV_EOS_SENT*/
{ S_NOP, S_XXX, S_XXX, S_XXX, S_XXX, S_CLS, S_CLN, S_NOP, },/* EV_EOS_SENT*/
{ S_NOP, S_XXX, S_CLS, S_XXX, S_XXX, S_CLS, S_XXX, S_XXX, },/* EV_IN_ERROR*/
};

Expand Down

0 comments on commit decce82

Please sign in to comment.