mirror of
https://github.com/cloudwu/skynet.git
synced 2026-07-24 20:23:06 +00:00
After connecting, socket can send before connected. remove block connect api
This commit is contained in:
@@ -133,12 +133,6 @@ skynet_socket_connect(struct skynet_context *ctx, const char *host, int port) {
|
||||
return socket_server_connect(SOCKET_SERVER, source, host, port);
|
||||
}
|
||||
|
||||
int
|
||||
skynet_socket_block_connect(struct skynet_context *ctx, const char *host, int port) {
|
||||
uint32_t source = skynet_context_handle(ctx);
|
||||
return socket_server_block_connect(SOCKET_SERVER, source, host, port);
|
||||
}
|
||||
|
||||
int
|
||||
skynet_socket_bind(struct skynet_context *ctx, int fd) {
|
||||
uint32_t source = skynet_context_handle(ctx);
|
||||
|
||||
@@ -25,7 +25,6 @@ int skynet_socket_send(struct skynet_context *ctx, int id, void *buffer, int sz)
|
||||
void skynet_socket_send_lowpriority(struct skynet_context *ctx, int id, void *buffer, int sz);
|
||||
int skynet_socket_listen(struct skynet_context *ctx, const char *host, int port, int backlog);
|
||||
int skynet_socket_connect(struct skynet_context *ctx, const char *host, int port);
|
||||
int skynet_socket_block_connect(struct skynet_context *ctx, const char *host, int port);
|
||||
int skynet_socket_bind(struct skynet_context *ctx, int fd);
|
||||
void skynet_socket_close(struct skynet_context *ctx, int id);
|
||||
void skynet_socket_start(struct skynet_context *ctx, int id);
|
||||
|
||||
@@ -147,6 +147,8 @@ reserve_id(struct socket_server *ss) {
|
||||
struct socket *s = &ss->slot[id % MAX_SOCKET];
|
||||
if (s->type == SOCKET_TYPE_INVALID) {
|
||||
if (__sync_bool_compare_and_swap(&s->type, SOCKET_TYPE_INVALID, SOCKET_TYPE_RESERVE)) {
|
||||
s->id = id;
|
||||
s->fd = -1;
|
||||
return id;
|
||||
} else {
|
||||
// retry
|
||||
@@ -287,7 +289,7 @@ new_fd(struct socket_server *ss, int id, int fd, uintptr_t opaque, bool add) {
|
||||
|
||||
// return -1 when connecting
|
||||
static int
|
||||
open_socket(struct socket_server *ss, struct request_open * request, struct socket_message *result, bool blocking) {
|
||||
open_socket(struct socket_server *ss, struct request_open * request, struct socket_message *result) {
|
||||
int id = request->id;
|
||||
result->opaque = request->opaque;
|
||||
result->id = id;
|
||||
@@ -316,18 +318,14 @@ open_socket(struct socket_server *ss, struct request_open * request, struct sock
|
||||
continue;
|
||||
}
|
||||
socket_keepalive(sock);
|
||||
if (!blocking) {
|
||||
sp_nonblocking(sock);
|
||||
}
|
||||
sp_nonblocking(sock);
|
||||
status = connect( sock, ai_ptr->ai_addr, ai_ptr->ai_addrlen);
|
||||
if ( status != 0 && errno != EINPROGRESS) {
|
||||
close(sock);
|
||||
sock = -1;
|
||||
continue;
|
||||
}
|
||||
if (blocking) {
|
||||
sp_nonblocking(sock);
|
||||
}
|
||||
sp_nonblocking(sock);
|
||||
break;
|
||||
}
|
||||
|
||||
@@ -512,7 +510,7 @@ send_socket(struct socket_server *ss, struct request_send * request, struct sock
|
||||
return -1;
|
||||
}
|
||||
assert(s->type != SOCKET_TYPE_PLISTEN && s->type != SOCKET_TYPE_LISTEN);
|
||||
if (send_buffer_empty(s)) {
|
||||
if (send_buffer_empty(s) && s->type == SOCKET_TYPE_CONNECTED) {
|
||||
int n = write(s->fd, request->buffer, request->sz);
|
||||
if (n<0) {
|
||||
switch(errno) {
|
||||
@@ -687,7 +685,7 @@ ctrl_cmd(struct socket_server *ss, struct socket_message *result) {
|
||||
case 'K':
|
||||
return close_socket(ss,(struct request_close *)buffer, result);
|
||||
case 'O':
|
||||
return open_socket(ss, (struct request_open *)buffer, result, false);
|
||||
return open_socket(ss, (struct request_open *)buffer, result);
|
||||
case 'X':
|
||||
result->opaque = 0;
|
||||
result->id = 0;
|
||||
@@ -765,7 +763,9 @@ report_connect(struct socket_server *ss, struct socket *s, struct socket_message
|
||||
result->opaque = s->opaque;
|
||||
result->id = s->id;
|
||||
result->ud = 0;
|
||||
sp_write(ss->event_fd, s->fd, s, false);
|
||||
if (send_buffer_empty(s)) {
|
||||
sp_write(ss->event_fd, s->fd, s, false);
|
||||
}
|
||||
union sockaddr_all u;
|
||||
socklen_t slen = sizeof(u);
|
||||
if (getpeername(s->fd, &u.s, &slen) == 0) {
|
||||
@@ -939,19 +939,6 @@ socket_server_connect(struct socket_server *ss, uintptr_t opaque, const char * a
|
||||
return request.u.open.id;
|
||||
}
|
||||
|
||||
int
|
||||
socket_server_block_connect(struct socket_server *ss, uintptr_t opaque, const char * addr, int port) {
|
||||
struct request_package request;
|
||||
struct socket_message result;
|
||||
open_request(ss, &request, opaque, addr, port);
|
||||
int ret = open_socket(ss, &request.u.open, &result, true);
|
||||
if (ret == SOCKET_OPEN) {
|
||||
return result.id;
|
||||
} else {
|
||||
return -1;
|
||||
}
|
||||
}
|
||||
|
||||
// return -1 when error
|
||||
int64_t
|
||||
socket_server_send(struct socket_server *ss, int id, const void * buffer, int sz) {
|
||||
@@ -959,7 +946,6 @@ socket_server_send(struct socket_server *ss, int id, const void * buffer, int sz
|
||||
if (s->id != id || s->type == SOCKET_TYPE_INVALID) {
|
||||
return -1;
|
||||
}
|
||||
assert(s->type != SOCKET_TYPE_RESERVE);
|
||||
|
||||
struct request_package request;
|
||||
request.u.send.id = id;
|
||||
@@ -976,7 +962,6 @@ socket_server_send_lowpriority(struct socket_server *ss, int id, const void * bu
|
||||
if (s->id != id || s->type == SOCKET_TYPE_INVALID) {
|
||||
return;
|
||||
}
|
||||
assert(s->type != SOCKET_TYPE_RESERVE);
|
||||
|
||||
struct request_package request;
|
||||
request.u.send.id = id;
|
||||
|
||||
@@ -36,6 +36,4 @@ int socket_server_listen(struct socket_server *, uintptr_t opaque, const char *
|
||||
int socket_server_connect(struct socket_server *, uintptr_t opaque, const char * addr, int port);
|
||||
int socket_server_bind(struct socket_server *, uintptr_t opaque, int fd);
|
||||
|
||||
int socket_server_block_connect(struct socket_server *, uintptr_t opaque, const char * addr, int port);
|
||||
|
||||
#endif
|
||||
|
||||
Reference in New Issue
Block a user