Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1303)

Unified Diff: net/quic/quic_server.cc

Issue 340433002: Port QuicServer to Chrome network stack (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Indentation fix Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/quic/quic_server.cc
diff --git a/net/tools/quic/quic_server.cc b/net/quic/quic_server.cc
similarity index 22%
copy from net/tools/quic/quic_server.cc
copy to net/quic/quic_server.cc
index be3d9fbd128178a8161294247535f455dd3053c6..6192779f2df609420178d862ce446e2e8e47ede9 100644
--- a/net/tools/quic/quic_server.cc
+++ b/net/quic/quic_server.cc
@@ -2,182 +2,117 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "net/tools/quic/quic_server.h"
+#include "net/quic/quic_server.h"
-#include <errno.h>
-#include <features.h>
-#include <netinet/in.h>
#include <string.h>
-#include <sys/epoll.h>
-#include <sys/socket.h>
#include "net/base/ip_endpoint.h"
+#include "net/base/net_errors.h"
#include "net/quic/congestion_control/tcp_receiver.h"
#include "net/quic/crypto/crypto_handshake.h"
#include "net/quic/crypto/quic_random.h"
-#include "net/quic/quic_clock.h"
#include "net/quic/quic_crypto_stream.h"
#include "net/quic/quic_data_reader.h"
+#include "net/quic/quic_dispatcher.h"
+#include "net/quic/quic_in_memory_cache.h"
#include "net/quic/quic_protocol.h"
-#include "net/tools/quic/quic_in_memory_cache.h"
-#include "net/tools/quic/quic_socket_utils.h"
+#include "net/quic/quic_server_packet_writer.h"
+#include "net/udp/udp_server_socket.h"
-#define MMSG_MORE 0
+namespace net {
-#ifndef SO_RXQ_OVFL
-#define SO_RXQ_OVFL 40
-#endif
+namespace {
-const int kEpollFlags = EPOLLIN | EPOLLOUT | EPOLLET;
-static const char kSourceAddressTokenSecret[] = "secret";
-const uint32 kServerInitialFlowControlWindow = 100 * net::kMaxPacketSize;
+const char kSourceAddressTokenSecret[] = "secret";
-namespace net {
-namespace tools {
-
-QuicServer::QuicServer()
- : port_(0),
- fd_(-1),
- packets_dropped_(0),
- overflow_supported_(false),
- use_recvmmsg_(false),
- crypto_config_(kSourceAddressTokenSecret, QuicRandom::GetInstance()),
- supported_versions_(QuicSupportedVersions()) {
- // Use hardcoded crypto parameters for now.
- config_.SetDefaults();
- Initialize();
-}
+// Allocate some extra space so we can send an error if the client goes over
+// the limit.
+const int kReadBufferSize = 2 * kMaxPacketSize;
+
+const uint32 kServerInitialFlowControlWindow = 100 * kMaxPacketSize;
+
+} // namespace
QuicServer::QuicServer(const QuicConfig& config,
const QuicVersionVector& supported_versions)
- : port_(0),
- fd_(-1),
- packets_dropped_(0),
- overflow_supported_(false),
- use_recvmmsg_(false),
+ : helper_(base::MessageLoop::current()->message_loop_proxy().get(),
+ &clock_,
+ QuicRandom::GetInstance()),
config_(config),
crypto_config_(kSourceAddressTokenSecret, QuicRandom::GetInstance()),
- supported_versions_(supported_versions) {
+ supported_versions_(supported_versions),
+ read_pending_(false),
+ synchronous_read_count_(0),
+ read_buffer_(new IOBufferWithSize(kReadBufferSize)),
+ weak_factory_(this) {
Initialize();
}
void QuicServer::Initialize() {
-#if MMSG_MORE
- use_recvmmsg_ = true;
-#endif
- epoll_server_.set_timeout_in_us(50 * 1000);
// Initialize the in memory cache now.
QuicInMemoryCache::GetInstance();
- QuicEpollClock clock(&epoll_server_);
-
scoped_ptr<CryptoHandshakeMessage> scfg(
- crypto_config_.AddDefaultConfig(
- QuicRandom::GetInstance(), &clock,
- QuicCryptoServerConfig::ConfigOptions()));
+ crypto_config_.AddDefaultConfig(helper_.GetRandomGenerator(),
+ helper_.GetClock(),
+ QuicCryptoServerConfig::ConfigOptions()));
- // Set flow control options in the config.
config_.SetInitialCongestionWindowToSend(kServerInitialFlowControlWindow);
}
QuicServer::~QuicServer() {
}
-bool QuicServer::Listen(const IPEndPoint& address) {
- port_ = address.port();
- int address_family = address.GetSockAddrFamily();
- fd_ = socket(address_family, SOCK_DGRAM | SOCK_NONBLOCK, IPPROTO_UDP);
- if (fd_ < 0) {
- LOG(ERROR) << "CreateSocket() failed: " << strerror(errno);
- return false;
- }
+int QuicServer::Listen(const IPEndPoint& address) {
+ scoped_ptr<UDPServerSocket> socket(
+ new UDPServerSocket(&net_log_, NetLog::Source()));
- int rc = QuicSocketUtils::SetGetAddressInfo(fd_, address_family);
+ socket->AllowAddressReuse();
+ int rc = socket->Listen(address);
if (rc < 0) {
- LOG(ERROR) << "IP detection not supported" << strerror(errno);
- return false;
- }
-
- int get_overflow = 1;
- rc = setsockopt(
- fd_, SOL_SOCKET, SO_RXQ_OVFL, &get_overflow, sizeof(get_overflow));
-
- if (rc < 0) {
- DLOG(WARNING) << "Socket overflow detection not supported";
- } else {
- overflow_supported_ = true;
+ LOG(ERROR) << "Listen() failed: " << ErrorToString(rc);
+ return rc;
}
// These send and receive buffer sizes are sized for a single connection,
// because the default usage of QuicServer is as a test server with one or
// two clients. Adjust higher for use with many clients.
- if (!QuicSocketUtils::SetReceiveBufferSize(fd_,
- TcpReceiver::kReceiveWindowTCP)) {
- return false;
- }
-
- if (!QuicSocketUtils::SetSendBufferSize(fd_,
- TcpReceiver::kReceiveWindowTCP)) {
- return false;
+ rc = socket->SetReceiveBufferSize(TcpReceiver::kReceiveWindowTCP);
+ if (rc < 0) {
+ LOG(ERROR) << "SetReceiveBufferSize() failed: " << ErrorToString(rc);
+ return rc;
}
- // Enable the socket option that allows the local address to be
- // returned if the socket is bound to more than on address.
- int get_local_ip = 1;
- rc = setsockopt(fd_, IPPROTO_IP, IP_PKTINFO,
- &get_local_ip, sizeof(get_local_ip));
- if (rc == 0 && address_family == AF_INET6) {
- rc = setsockopt(fd_, IPPROTO_IPV6, IPV6_RECVPKTINFO,
- &get_local_ip, sizeof(get_local_ip));
- }
- if (rc != 0) {
- LOG(ERROR) << "Failed to set required socket options";
- return false;
+ rc = socket->SetSendBufferSize(20 * kMaxPacketSize);
+ if (rc < 0) {
+ LOG(ERROR) << "SetSendBufferSize() failed: " << ErrorToString(rc);
+ return rc;
}
- sockaddr_storage raw_addr;
- socklen_t raw_addr_len = sizeof(raw_addr);
- CHECK(address.ToSockAddr(reinterpret_cast<sockaddr*>(&raw_addr),
- &raw_addr_len));
- rc = bind(fd_,
- reinterpret_cast<const sockaddr*>(&raw_addr),
- sizeof(raw_addr));
+ rc = socket->GetLocalAddress(&server_address_);
if (rc < 0) {
- LOG(ERROR) << "Bind failed: " << strerror(errno);
- return false;
+ LOG(ERROR) << "GetLocalAddress() failed: " << ErrorToString(rc);
+ return rc;
}
- DVLOG(1) << "Listening on " << address.ToString();
- if (port_ == 0) {
- SockaddrStorage storage;
- IPEndPoint server_address;
- if (getsockname(fd_, storage.addr, &storage.addr_len) != 0 ||
- !server_address.FromSockAddr(storage.addr, storage.addr_len)) {
- LOG(ERROR) << "Unable to get self address. Error: " << strerror(errno);
- return false;
- }
- port_ = server_address.port();
- DVLOG(1) << "Kernel assigned port is " << port_;
- }
+ DVLOG(1) << "Listening on " << server_address_.ToString();
- epoll_server_.RegisterFD(fd_, this, kEpollFlags);
- dispatcher_.reset(CreateQuicDispatcher());
- dispatcher_->Initialize(fd_);
+ socket_.swap(socket);
- return true;
-}
+ dispatcher_.reset(
+ new QuicDispatcher(config_,
+ crypto_config_,
+ supported_versions_,
+ &helper_));
+ QuicServerPacketWriter* writer = new QuicServerPacketWriter(
+ socket_.get(),
+ dispatcher_.get());
+ dispatcher_->Initialize(writer);
-QuicDispatcher* QuicServer::CreateQuicDispatcher() {
- return new QuicDispatcher(
- config_,
- crypto_config_,
- supported_versions_,
- &epoll_server_);
-}
+ StartReading();
-void QuicServer::WaitForEvents() {
- epoll_server_.WaitForEventsAndExecuteCallbacks();
+ return OK;
}
void QuicServer::Shutdown() {
@@ -185,60 +120,56 @@ void QuicServer::Shutdown() {
// notify clients that they're closing.
dispatcher_->Shutdown();
- close(fd_);
- fd_ = -1;
+ socket_->Close();
+ socket_.reset();
}
-void QuicServer::OnEvent(int fd, EpollEvent* event) {
- DCHECK_EQ(fd, fd_);
- event->out_ready_mask = 0;
-
- if (event->in_events & EPOLLIN) {
- DVLOG(1) << "EPOLLIN";
- bool read = true;
- while (read) {
- read = ReadAndDispatchSinglePacket(
- fd_, port_, dispatcher_.get(),
- overflow_supported_ ? &packets_dropped_ : NULL);
- }
+void QuicServer::StartReading() {
+ if (read_pending_) {
+ return;
}
- if (event->in_events & EPOLLOUT) {
- dispatcher_->OnCanWrite();
- if (dispatcher_->HasPendingWrites()) {
- event->out_ready_mask |= EPOLLOUT;
- }
+ read_pending_ = true;
+
+ int result = socket_->RecvFrom(
+ read_buffer_.get(),
+ read_buffer_->size(),
+ &client_address_,
+ base::Bind(&QuicServer::OnReadComplete, base::Unretained(this)));
+
+ if (result == ERR_IO_PENDING) {
+ synchronous_read_count_ = 0;
+ return;
}
- if (event->in_events & EPOLLERR) {
+
+ if (++synchronous_read_count_ > 32) {
+ synchronous_read_count_ = 0;
+ // Schedule the processing through the message loop to 1) prevent infinite
+ // recursion and 2) avoid blocking the thread for too long.
+ base::MessageLoop::current()->PostTask(
+ FROM_HERE,
+ base::Bind(&QuicServer::OnReadComplete,
+ weak_factory_.GetWeakPtr(),
+ result));
+ } else {
+ OnReadComplete(result);
}
}
-/* static */
-bool QuicServer::ReadAndDispatchSinglePacket(int fd,
- int port,
- QuicDispatcher* dispatcher,
- uint32* packets_dropped) {
- // Allocate some extra space so we can send an error if the client goes over
- // the limit.
- char buf[2 * kMaxPacketSize];
-
- IPEndPoint client_address;
- IPAddressNumber server_ip;
- int bytes_read =
- QuicSocketUtils::ReadPacket(fd, buf, arraysize(buf),
- packets_dropped,
- &server_ip, &client_address);
-
- if (bytes_read < 0) {
- return false; // We failed to read.
- }
+void QuicServer::OnReadComplete(int result) {
+ read_pending_ = false;
+ if (result == 0)
+ result = ERR_CONNECTION_CLOSED;
- QuicEncryptedPacket packet(buf, bytes_read, false);
+ if (result < 0) {
+ LOG(ERROR) << "QuicServer read failed: " << ErrorToString(result);
+ Shutdown();
+ return;
+ }
- IPEndPoint server_address(server_ip, port);
- dispatcher->ProcessPacket(server_address, client_address, packet);
+ QuicEncryptedPacket packet(read_buffer_->data(), result, false);
+ dispatcher_->ProcessPacket(server_address_, client_address_, packet);
- return true;
+ StartReading();
}
-} // namespace tools
} // namespace net

Powered by Google App Engine
This is Rietveld 408576698