New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 690095 link

Starred by 1 user
Status: WontFix
Owner:
█ xunji...@chromium.org
Last visit > 30 days ago
Closed: Aug 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 620852



Sign in to add a comment

MDnsConnection::SocketHandler keeps 9KB IOBuffers around

Project Member Reported by xunji...@chromium.org, Feb 8 2017 
MDnsConnection::SocketHandler allocates a 9KB IOBuffer and uses it to read from a datagram socket. MDnsClient owns a vector of these socket handlers. Each has a 9KB buffer. These handlers do not free up the read buffers when done reading, so these read buffers can be kept alive for a while.

Comment 1 by xunji...@chromium.org, Feb 8 2017

Summary: MDnsConnection::SocketHandler keeps 9KB IOBuffers around (was: MDnsConnection::SocketHandler keeps unneeded 9KB IOBuffers around)
Hmm... actually that isn't the cause. I tried https://codereview.chromium.org/2684113002/, but it didn't improve things. Looks like MDnsConnection is reading from the udp socket. The udp socket continuously delivers packets of ~40 bytes each. The IOBuffer probably doesn't need to be 9KB. But I don't know who is still working on MDns.

Comment 2 by benhenry@chromium.org, Aug 7 2017 

Is this still a Pri-3? Are you planning to get to this work in the knowable future?

Comment 3 by xunji...@chromium.org, Aug 7 2017

Status: WontFix (was: Assigned)
Archive this one. This is probably not worth pursuing given the amount of buffer usage.
►

Sign in to add a comment