Skip to content

Reply Copy Avoidance #14608

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
sundb wants to merge 31 commits into
base: unstable
Choose a base branch
from
Open

Reply Copy Avoidance #14608

Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
e4bf61f
Reply Copy Avoidance
sundb Dec 8, 2025
f9fd7e6
Fix logreqres
sundb Dec 8, 2025
b5206dd
Fix comparsion
sundb Dec 8, 2025
c2686de
Refine code
sundb Dec 8, 2025
adfe25a
Fix logreqres
sundb Dec 8, 2025
7a58104
calculate net_output_bytes_curr_cmd correctly
sundb Dec 9, 2025
8e2ba28
Add debug switch for reply copy avoidance
sundb Dec 9, 2025
43ff864
fix test failed
sundb Dec 9, 2025
69c0ded
Replace config with macro
sundb Dec 9, 2025
93c09f9
Change the position of debug REPLY-COPY-AVOIDANCE
sundb Dec 9, 2025
ebc761f
Add commment for why we cant resize c->buf in cron
sundb Dec 9, 2025
f9b00e1
Update comment
sundb Dec 9, 2025
61f4b87
Update comment
sundb Dec 9, 2025
8fe1715
Merge remote-tracking branch 'origin/unstable' into reply-copy-avoid-1
sundb Dec 9, 2025
852beaf
Change the type of refcount to unsigned short
sundb Dec 10, 2025
5b9f766
Fix the type of refcount
sundb Dec 10, 2025
01f2f93
Add space
sundb Dec 10, 2025
bceddd5
enable prefetch even with reply copy avoidance
sundb Dec 12, 2025
ceff395
format
sundb Dec 22, 2025
2f36e7d
change expected_refcount to unsigned long for activeDefragStringObEx()
sundb Dec 22, 2025
6329d5b
Add assert to avoid refcount exceeding the max value
sundb Dec 22, 2025
517efcc
Update src/networking.c
sundb Dec 24, 2025
b352a54
code review
sundb Dec 24, 2025
cb0b483
Merge remote-tracking branch 'origin/unstable' into reply-copy-avoid-1
sundb Dec 29, 2025
46b71ec
package the refcount with expirable and iskvobj
sundb Jan 7, 2026
c6d6847
Fix build complain for old build chain
sundb Jan 7, 2026
66c1d23
Merge remote-tracking branch 'origin/unstable' into reply-copy-avoid-1
sundb Jan 7, 2026
28eccce
Fix complain
sundb Jan 7, 2026
569d8a9
Fix typo
sundb Jan 7, 2026
0574791
Refine
sundb Jan 7, 2026
de80ea8
don't use release get for refcount
sundb Jan 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update comment
  • Loading branch information
@sundb
sundb committed Dec 9, 2025
commit 61f4b87104e4f3fae2d20213dffaa602cea1abf1
3 changes: 1 addition & 2 deletions src/networking.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2227,8 +2227,7 @@ static int replyIOVReachLimit(ReplyIOV *reply_iov) {
return reply_iov->iovcnt >= reply_iov->iovmax || reply_iov->iov_bytes_len >= NET_MAX_WRITES_PER_EVENT;
}

/* Helper function to process encoded buffer and build iov array.
* Uses last_header and sentlen to track position for partial sends. */
/* Helper function to process encoded buffer and build iov array. */
static void processEncodedBufferForWrite(ReplyIOV *reply_iov, char *start_ptr, char *end_ptr, size_t offset) {
char *ptr = start_ptr;
while (ptr < end_ptr && !replyIOVReachLimit(reply_iov)) {
Expand Down
Loading