libs/rt-thread
libs
/
rt-thread
4
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[net][sal][netdev] Delete the link_up status judgment in the send/recv interface 

Signed-off-by: chenyong <1521761801@qq.com>
This commit is contained in:
chenyong 2019-05-10 15:57:58 +08:00
parent 9eef644169
commit 4de0533a51
3 changed files with 28 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
components/net/netdev/include/netdev.h
View File

@ -133,7 +133,7 @@ int netdev_register(struct netdev *netdev, const char *name, void *user_data);
int netdev_unregister(struct netdev *netdev); int netdev_unregister(struct netdev *netdev);
/* Get network interface device object */ /* Get network interface device object */
struct netdev *netdev_get_first_link_up(void); struct netdev *netdev_get_first_by_flags(uint16_t flags);
struct netdev *netdev_get_by_ipaddr(ip_addr_t *ip_addr); struct netdev *netdev_get_by_ipaddr(ip_addr_t *ip_addr);
struct netdev *netdev_get_by_name(const char *name); struct netdev *netdev_get_by_name(const char *name);
#ifdef RT_USING_SAL #ifdef RT_USING_SAL

12
components/net/netdev/src/netdev.c
View File

@ -134,12 +134,14 @@ int netdev_unregister(struct netdev *netdev)
/** /**
* This function will get the first network interface device * This function will get the first network interface device
* with the link_up status in network interface device list. * with the flags in network interface device list.
*
* @param flags the network interface device flags
* *
* @return != NULL: network interface device object * @return != NULL: network interface device object
* NULL: get failed * NULL: get failed
*/ */
struct netdev *netdev_get_first_link_up(void) struct netdev *netdev_get_first_by_flags(uint16_t flags)
{ {
rt_base_t level; rt_base_t level;
rt_slist_t *node = RT_NULL; rt_slist_t *node = RT_NULL;
@ -155,7 +157,7 @@ struct netdev *netdev_get_first_link_up(void)
for (node = &(netdev_list->list); node; node = rt_slist_next(node)) for (node = &(netdev_list->list); node; node = rt_slist_next(node))
{ {
netdev = rt_slist_entry(node, struct netdev, list); netdev = rt_slist_entry(node, struct netdev, list);
if (netdev && netdev_is_up(netdev) && netdev_is_link_up(netdev)) if (netdev && (netdev->flags & flags) != 0)
{ {
rt_hw_interrupt_enable(level); rt_hw_interrupt_enable(level);
return netdev; return netdev;
@ -679,7 +681,7 @@ static void netdev_auto_change_default(struct netdev *netdev)
if (rt_memcmp(netdev, netdev_default, sizeof(struct netdev)) == 0) if (rt_memcmp(netdev, netdev_default, sizeof(struct netdev)) == 0)
{ {
new_netdev = netdev_get_first_link_up(); new_netdev = netdev_get_first_by_flags(NETDEV_FLAG_LINK_UP);
if (new_netdev) if (new_netdev)
{ {
netdev_set_default(new_netdev); netdev_set_default(new_netdev);
@ -964,7 +966,7 @@ static int netdev_cmd_ping(char* target_name, rt_uint32_t times, rt_size_t size)
else else
{ {
/* using first internet up status network interface device */ /* using first internet up status network interface device */
netdev = netdev_get_first_link_up(); netdev = netdev_get_first_by_flags(NETDEV_FLAG_LINK_UP);
if (netdev == RT_NULL || NETDEV_PING_IS_COMMONICABLE(netdev) == 0) if (netdev == RT_NULL || NETDEV_PING_IS_COMMONICABLE(netdev) == 0)
{ {
rt_kprintf("ping: network interface device get error.\n"); rt_kprintf("ping: network interface device get error.\n");

40
components/net/sal_socket/src/sal_socket.c
View File

@ -70,9 +70,9 @@ do {
} \ } \
}while(0) \ }while(0) \
#define SAL_NETDEV_IS_COMMONICABLE(netdev) \ #define SAL_NETDEV_IS_UP(netdev) \
do { \ do { \
if (!netdev_is_up(netdev) || !netdev_is_link_up(netdev)){ \ if (!netdev_is_up(netdev)) { \
return -1; \ return -1; \
} \ } \
}while(0) \ }while(0) \
@ -86,9 +86,9 @@ do {
}while(0) \ }while(0) \
#define SAL_NETDEV_NETDBOPS_VALID(netdev, pf, ops) \ #define SAL_NETDEV_NETDBOPS_VALID(netdev, pf, ops) \
((netdev) && netdev_is_up(netdev) && netdev_is_link_up(netdev) && \ ((netdev) && netdev_is_up(netdev) && \
((pf) = (struct sal_proto_family *) (netdev)->sal_user_data) != RT_NULL && \ ((pf) = (struct sal_proto_family *) (netdev)->sal_user_data) != RT_NULL && \
(pf)->netdb_ops->ops) \ (pf)->netdb_ops->ops) \
/** /**
* SAL (Socket Abstraction Layer) initialize. * SAL (Socket Abstraction Layer) initialize.
@ -748,8 +748,8 @@ int sal_connect(int socket, const struct sockaddr *name, socklen_t namelen)
/* get the socket object by socket descriptor */ /* get the socket object by socket descriptor */
SAL_SOCKET_OBJ_GET(sock, socket); SAL_SOCKET_OBJ_GET(sock, socket);
/* check the network interface is commonicable */ /* check the network interface is up status */
SAL_NETDEV_IS_COMMONICABLE(sock->netdev); SAL_NETDEV_IS_UP(sock->netdev);
/* check the network interface socket opreation */ /* check the network interface socket opreation */
SAL_NETDEV_SOCKETOPS_VALID(sock->netdev, pf, connect); SAL_NETDEV_SOCKETOPS_VALID(sock->netdev, pf, connect);
@ -792,8 +792,8 @@ int sal_recvfrom(int socket, void *mem, size_t len, int flags,
/* get the socket object by socket descriptor */ /* get the socket object by socket descriptor */
SAL_SOCKET_OBJ_GET(sock, socket); SAL_SOCKET_OBJ_GET(sock, socket);
/* check the network interface is commonicable */ /* check the network interface is up status */
SAL_NETDEV_IS_COMMONICABLE(sock->netdev); SAL_NETDEV_IS_UP(sock->netdev);
/* check the network interface socket opreation */ /* check the network interface socket opreation */
SAL_NETDEV_SOCKETOPS_VALID(sock->netdev, pf, recvfrom); SAL_NETDEV_SOCKETOPS_VALID(sock->netdev, pf, recvfrom);
@ -826,8 +826,8 @@ int sal_sendto(int socket, const void *dataptr, size_t size, int flags,
/* get the socket object by socket descriptor */ /* get the socket object by socket descriptor */
SAL_SOCKET_OBJ_GET(sock, socket); SAL_SOCKET_OBJ_GET(sock, socket);
/* check the network interface is commonicable */ /* check the network interface is up status */
SAL_NETDEV_IS_COMMONICABLE(sock->netdev); SAL_NETDEV_IS_UP(sock->netdev);
/* check the network interface socket opreation */ /* check the network interface socket opreation */
SAL_NETDEV_SOCKETOPS_VALID(sock->netdev, pf, sendto); SAL_NETDEV_SOCKETOPS_VALID(sock->netdev, pf, sendto);
@ -965,8 +965,8 @@ int sal_poll(struct dfs_fd *file, struct rt_pollreq *req)
/* get the socket object by socket descriptor */ /* get the socket object by socket descriptor */
SAL_SOCKET_OBJ_GET(sock, socket); SAL_SOCKET_OBJ_GET(sock, socket);
/* check the network interface is commonicable */ /* check the network interface is up status */
SAL_NETDEV_IS_COMMONICABLE(sock->netdev); SAL_NETDEV_IS_UP(sock->netdev);
/* check the network interface socket opreation */ /* check the network interface socket opreation */
SAL_NETDEV_SOCKETOPS_VALID(sock->netdev, pf, poll); SAL_NETDEV_SOCKETOPS_VALID(sock->netdev, pf, poll);
@ -985,8 +985,8 @@ struct hostent *sal_gethostbyname(const char *name)
} }
else else
{ {
/* get the first network interface device with the link up status */ /* get the first network interface device with up status */
netdev = netdev_get_first_link_up(); netdev = netdev_get_first_by_flags(NETDEV_FLAG_UP);
if (SAL_NETDEV_NETDBOPS_VALID(netdev, pf, gethostbyname)) if (SAL_NETDEV_NETDBOPS_VALID(netdev, pf, gethostbyname))
{ {
return pf->netdb_ops->gethostbyname(name); return pf->netdb_ops->gethostbyname(name);
@ -1008,8 +1008,8 @@ int sal_gethostbyname_r(const char *name, struct hostent *ret, char *buf,
} }
else else
{ {
/* get the first network interface device with the link up status */ /* get the first network interface device with up status */
netdev = netdev_get_first_link_up(); netdev = netdev_get_first_by_flags(NETDEV_FLAG_UP);
if (SAL_NETDEV_NETDBOPS_VALID(netdev, pf, gethostbyname_r)) if (SAL_NETDEV_NETDBOPS_VALID(netdev, pf, gethostbyname_r))
{ {
return pf->netdb_ops->gethostbyname_r(name, ret, buf, buflen, result, h_errnop); return pf->netdb_ops->gethostbyname_r(name, ret, buf, buflen, result, h_errnop);
@ -1033,8 +1033,8 @@ int sal_getaddrinfo(const char *nodename,
} }
else else
{ {
/* get the first network interface device with the link up status */ /* get the first network interface device with up status */
netdev = netdev_get_first_link_up(); netdev = netdev_get_first_by_flags(NETDEV_FLAG_UP);
if (SAL_NETDEV_NETDBOPS_VALID(netdev, pf, getaddrinfo)) if (SAL_NETDEV_NETDBOPS_VALID(netdev, pf, getaddrinfo))
{ {
return pf->netdb_ops->getaddrinfo(nodename, servname, hints, res); return pf->netdb_ops->getaddrinfo(nodename, servname, hints, res);
@ -1055,8 +1055,8 @@ void sal_freeaddrinfo(struct addrinfo *ai)
} }
else else
{ {
/* get the first network interface device with the link up status */ /* get the first network interface device with up status */
netdev = netdev_get_first_link_up(); netdev = netdev_get_first_by_flags(NETDEV_FLAG_UP);
if (SAL_NETDEV_NETDBOPS_VALID(netdev, pf, freeaddrinfo)) if (SAL_NETDEV_NETDBOPS_VALID(netdev, pf, freeaddrinfo))
{ {
pf->netdb_ops->freeaddrinfo(ai); pf->netdb_ops->freeaddrinfo(ai);