Changeset b7fd2a0 in mainline for uspace/srv/net/ethip


Ignore:
Timestamp:
2018-01-13T03:10:29Z (8 years ago)
Author:
Jiří Zárevúcky <zarevucky.jiri@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
a53ed3a
Parents:
36f0738
Message:

Use errno_t in all uspace and kernel code.

Change type of every variable, parameter and return value that holds an
<errno.h> constant to either errno_t (the usual case), or sys_errno_t
(some places in kernel). This is for the purpose of self-documentation,
as well as for type-checking with a bit of type definition hackery.

Although this is a massive commit, it is a simple text replacement, and thus
is very easy to verify. Simply do the following:

`
git checkout <this commit's hash>
git reset HEAD
git add .
tools/srepl '\berrno_t\b' int
git add .
tools/srepl '\bsys_errno_t\b' sysarg_t
git reset
git diff
`

While this doesn't ensure that the replacements are correct, it does ensure
that the commit doesn't do anything except those replacements. Since errno_t
is typedef'd to int in the usual case (and sys_errno_t to sysarg_t), even if
incorrect, this commit cannot change behavior.

Location:
uspace/srv/net/ethip
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • uspace/srv/net/ethip/arp.c

    r36f0738 rb7fd2a0  
    5050#define ARP_REQUEST_TIMEOUT (3 * 1000 * 1000)
    5151
    52 static int arp_send_packet(ethip_nic_t *nic, arp_eth_packet_t *packet);
     52static errno_t arp_send_packet(ethip_nic_t *nic, arp_eth_packet_t *packet);
    5353
    5454void arp_received(ethip_nic_t *nic, eth_frame_t *frame)
    … …  
    5757       
    5858        arp_eth_packet_t packet;
    59         int rc = arp_pdu_decode(frame->data, frame->size, &packet);
     59        errno_t rc = arp_pdu_decode(frame->data, frame->size, &packet);
    6060        if (rc != EOK)
    6161                return;
    … …  
    9494}
    9595
    96 int arp_translate(ethip_nic_t *nic, addr32_t src_addr, addr32_t ip_addr,
     96errno_t arp_translate(ethip_nic_t *nic, addr32_t src_addr, addr32_t ip_addr,
    9797    addr48_t mac_addr)
    9898{
    … …  
    103103        }
    104104
    105         int rc = atrans_lookup(ip_addr, mac_addr);
     105        errno_t rc = atrans_lookup(ip_addr, mac_addr);
    106106        if (rc == EOK)
    107107                return EOK;
    … …  
    122122}
    123123
    124 static int arp_send_packet(ethip_nic_t *nic, arp_eth_packet_t *packet)
     124static errno_t arp_send_packet(ethip_nic_t *nic, arp_eth_packet_t *packet)
    125125{
    126         int rc;
     126        errno_t rc;
    127127        void *pdata;
    128128        size_t psize;

  • uspace/srv/net/ethip/arp.h

    r36f0738 rb7fd2a0  
    4343
    4444extern void arp_received(ethip_nic_t *, eth_frame_t *);
    45 extern int arp_translate(ethip_nic_t *, addr32_t, addr32_t, addr48_t);
     45extern errno_t arp_translate(ethip_nic_t *, addr32_t, addr32_t, addr48_t);
    4646
    4747#endif

  • uspace/srv/net/ethip/atrans.c

    r36f0738 rb7fd2a0  
    5959}
    6060
    61 int atrans_add(addr32_t ip_addr, addr48_t mac_addr)
     61errno_t atrans_add(addr32_t ip_addr, addr48_t mac_addr)
    6262{
    6363        ethip_atrans_t *atrans;
    … …  
    8585}
    8686
    87 int atrans_remove(addr32_t ip_addr)
     87errno_t atrans_remove(addr32_t ip_addr)
    8888{
    8989        ethip_atrans_t *atrans;
    … …  
    103103}
    104104
    105 static int atrans_lookup_locked(addr32_t ip_addr, addr48_t mac_addr)
     105static errno_t atrans_lookup_locked(addr32_t ip_addr, addr48_t mac_addr)
    106106{
    107107        ethip_atrans_t *atrans = atrans_find(ip_addr);
    … …  
    113113}
    114114
    115 int atrans_lookup(addr32_t ip_addr, addr48_t mac_addr)
     115errno_t atrans_lookup(addr32_t ip_addr, addr48_t mac_addr)
    116116{
    117         int rc;
     117        errno_t rc;
    118118
    119119        fibril_mutex_lock(&atrans_list_lock);
    … …  
    134134}
    135135
    136 int atrans_lookup_timeout(addr32_t ip_addr, suseconds_t timeout,
     136errno_t atrans_lookup_timeout(addr32_t ip_addr, suseconds_t timeout,
    137137    addr48_t mac_addr)
    138138{
    139139        fibril_timer_t *t;
    140140        bool timedout;
    141         int rc;
     141        errno_t rc;
    142142
    143143        t = fibril_timer_create(NULL);

  • uspace/srv/net/ethip/atrans.h

    r36f0738 rb7fd2a0  
    4242#include "ethip.h"
    4343
    44 extern int atrans_add(addr32_t, addr48_t);
    45 extern int atrans_remove(addr32_t);
    46 extern int atrans_lookup(addr32_t, addr48_t);
    47 extern int atrans_lookup_timeout(addr32_t, suseconds_t, addr48_t);
     44extern errno_t atrans_add(addr32_t, addr48_t);
     45extern errno_t atrans_remove(addr32_t);
     46extern errno_t atrans_lookup(addr32_t, addr48_t);
     47extern errno_t atrans_lookup_timeout(addr32_t, suseconds_t, addr48_t);
    4848
    4949#endif

  • uspace/srv/net/ethip/ethip.c

    r36f0738 rb7fd2a0  
    5353#define NAME "ethip"
    5454
    55 static int ethip_open(iplink_srv_t *srv);
    56 static int ethip_close(iplink_srv_t *srv);
    57 static int ethip_send(iplink_srv_t *srv, iplink_sdu_t *sdu);
    58 static int ethip_send6(iplink_srv_t *srv, iplink_sdu6_t *sdu);
    59 static int ethip_get_mtu(iplink_srv_t *srv, size_t *mtu);
    60 static int ethip_get_mac48(iplink_srv_t *srv, addr48_t *mac);
    61 static int ethip_set_mac48(iplink_srv_t *srv, addr48_t *mac);
    62 static int ethip_addr_add(iplink_srv_t *srv, inet_addr_t *addr);
    63 static int ethip_addr_remove(iplink_srv_t *srv, inet_addr_t *addr);
     55static errno_t ethip_open(iplink_srv_t *srv);
     56static errno_t ethip_close(iplink_srv_t *srv);
     57static errno_t ethip_send(iplink_srv_t *srv, iplink_sdu_t *sdu);
     58static errno_t ethip_send6(iplink_srv_t *srv, iplink_sdu6_t *sdu);
     59static errno_t ethip_get_mtu(iplink_srv_t *srv, size_t *mtu);
     60static errno_t ethip_get_mac48(iplink_srv_t *srv, addr48_t *mac);
     61static errno_t ethip_set_mac48(iplink_srv_t *srv, addr48_t *mac);
     62static errno_t ethip_addr_add(iplink_srv_t *srv, inet_addr_t *addr);
     63static errno_t ethip_addr_remove(iplink_srv_t *srv, inet_addr_t *addr);
    6464
    6565static void ethip_client_conn(ipc_callid_t iid, ipc_call_t *icall, void *arg);
    … …  
    7777};
    7878
    79 static int ethip_init(void)
     79static errno_t ethip_init(void)
    8080{
    8181        async_set_fallback_port_handler(ethip_client_conn, NULL);
    8282       
    83         int rc = loc_server_register(NAME);
     83        errno_t rc = loc_server_register(NAME);
    8484        if (rc != EOK) {
    8585                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed registering server.");
    … …  
    9494}
    9595
    96 int ethip_iplink_init(ethip_nic_t *nic)
    97 {
    98         int rc;
     96errno_t ethip_iplink_init(ethip_nic_t *nic)
     97{
     98        errno_t rc;
    9999        service_id_t sid;
    100100        category_id_t iplink_cat;
    … …  
    158158}
    159159
    160 static int ethip_open(iplink_srv_t *srv)
     160static errno_t ethip_open(iplink_srv_t *srv)
    161161{
    162162        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_open()");
    … …  
    164164}
    165165
    166 static int ethip_close(iplink_srv_t *srv)
     166static errno_t ethip_close(iplink_srv_t *srv)
    167167{
    168168        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_close()");
    … …  
    170170}
    171171
    172 static int ethip_send(iplink_srv_t *srv, iplink_sdu_t *sdu)
     172static errno_t ethip_send(iplink_srv_t *srv, iplink_sdu_t *sdu)
    173173{
    174174        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_send()");
    … …  
    177177        eth_frame_t frame;
    178178       
    179         int rc = arp_translate(nic, sdu->src, sdu->dest, frame.dest);
     179        errno_t rc = arp_translate(nic, sdu->src, sdu->dest, frame.dest);
    180180        if (rc != EOK) {
    181181                log_msg(LOG_DEFAULT, LVL_WARN, "Failed to look up IPv4 address 0x%"
    … …  
    201201}
    202202
    203 static int ethip_send6(iplink_srv_t *srv, iplink_sdu6_t *sdu)
     203static errno_t ethip_send6(iplink_srv_t *srv, iplink_sdu6_t *sdu)
    204204{
    205205        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_send6()");
    … …  
    216216        void *data;
    217217        size_t size;
    218         int rc = eth_pdu_encode(&frame, &data, &size);
     218        errno_t rc = eth_pdu_encode(&frame, &data, &size);
    219219        if (rc != EOK)
    220220                return rc;
    … …  
    226226}
    227227
    228 int ethip_received(iplink_srv_t *srv, void *data, size_t size)
     228errno_t ethip_received(iplink_srv_t *srv, void *data, size_t size)
    229229{
    230230        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_received(): srv=%p", srv);
    … …  
    234234       
    235235        eth_frame_t frame;
    236         int rc = eth_pdu_decode(data, size, &frame);
     236        errno_t rc = eth_pdu_decode(data, size, &frame);
    237237        if (rc != EOK) {
    238238                log_msg(LOG_DEFAULT, LVL_DEBUG, " - eth_pdu_decode failed");
    … …  
    269269}
    270270
    271 static int ethip_get_mtu(iplink_srv_t *srv, size_t *mtu)
     271static errno_t ethip_get_mtu(iplink_srv_t *srv, size_t *mtu)
    272272{
    273273        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_get_mtu()");
    … …  
    276276}
    277277
    278 static int ethip_get_mac48(iplink_srv_t *srv, addr48_t *mac)
     278static errno_t ethip_get_mac48(iplink_srv_t *srv, addr48_t *mac)
    279279{
    280280        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_get_mac48()");
    … …  
    286286}
    287287
    288 static int ethip_set_mac48(iplink_srv_t *srv, addr48_t *mac)
     288static errno_t ethip_set_mac48(iplink_srv_t *srv, addr48_t *mac)
    289289{
    290290        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_set_mac48()");
    … …  
    296296}
    297297
    298 static int ethip_addr_add(iplink_srv_t *srv, inet_addr_t *addr)
     298static errno_t ethip_addr_add(iplink_srv_t *srv, inet_addr_t *addr)
    299299{
    300300        ethip_nic_t *nic = (ethip_nic_t *) srv->arg;
    … …  
    303303}
    304304
    305 static int ethip_addr_remove(iplink_srv_t *srv, inet_addr_t *addr)
     305static errno_t ethip_addr_remove(iplink_srv_t *srv, inet_addr_t *addr)
    306306{
    307307        ethip_nic_t *nic = (ethip_nic_t *) srv->arg;
    … …  
    312312int main(int argc, char *argv[])
    313313{
    314         int rc;
     314        errno_t rc;
    315315
    316316        printf(NAME ": HelenOS IP over Ethernet service\n");

  • uspace/srv/net/ethip/ethip.h

    r36f0738 rb7fd2a0  
    116116} ethip_atrans_t;
    117117
    118 extern int ethip_iplink_init(ethip_nic_t *);
    119 extern int ethip_received(iplink_srv_t *, void *, size_t);
     118extern errno_t ethip_iplink_init(ethip_nic_t *);
     119extern errno_t ethip_received(iplink_srv_t *, void *, size_t);
    120120
    121121#endif

  • uspace/srv/net/ethip/ethip_nic.c

    r36f0738 rb7fd2a0  
    5151#include "pdu.h"
    5252
    53 static int ethip_nic_open(service_id_t sid);
     53static errno_t ethip_nic_open(service_id_t sid);
    5454static void ethip_nic_cb_conn(ipc_callid_t iid, ipc_call_t *icall, void *arg);
    5555
    … …  
    5757static FIBRIL_MUTEX_INITIALIZE(ethip_discovery_lock);
    5858
    59 static int ethip_nic_check_new(void)
     59static errno_t ethip_nic_check_new(void)
    6060{
    6161        bool already_known;
    … …  
    6363        service_id_t *svcs;
    6464        size_t count, i;
    65         int rc;
     65        errno_t rc;
    6666
    6767        fibril_mutex_lock(&ethip_discovery_lock);
    … …  
    147147}
    148148
    149 static int ethip_nic_open(service_id_t sid)
     149static errno_t ethip_nic_open(service_id_t sid)
    150150{
    151151        bool in_list = false;
    … …  
    157157                return ENOMEM;
    158158       
    159         int rc = loc_service_get_name(sid, &nic->svc_name);
     159        errno_t rc = loc_service_get_name(sid, &nic->svc_name);
    160160        if (rc != EOK) {
    161161                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed getting service name.");
    … …  
    234234        uint8_t *addr;
    235235        size_t size;
    236         int rc;
     236        errno_t rc;
    237237
    238238        rc = async_data_write_accept((void **)&addr, false, 0, 0, 0, &size);
    … …  
    261261    ipc_call_t *call)
    262262{
    263         int rc;
     263        errno_t rc;
    264264        void *data;
    265265        size_t size;
    … …  
    324324}
    325325
    326 int ethip_nic_discovery_start(void)
    327 {
    328         int rc = loc_register_cat_change_cb(ethip_nic_cat_change_cb);
     326errno_t ethip_nic_discovery_start(void)
     327{
     328        errno_t rc = loc_register_cat_change_cb(ethip_nic_cat_change_cb);
    329329        if (rc != EOK) {
    330330                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed registering callback for NIC "
    … …  
    353353}
    354354
    355 int ethip_nic_send(ethip_nic_t *nic, void *data, size_t size)
    356 {
    357         int rc;
     355errno_t ethip_nic_send(ethip_nic_t *nic, void *data, size_t size)
     356{
     357        errno_t rc;
    358358        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_nic_send(size=%zu)", size);
    359359        rc = nic_send_frame(nic->sess, data, size);
    … …  
    368368 *
    369369 */
    370 static int ethip_nic_setup_multicast(ethip_nic_t *nic)
     370static errno_t ethip_nic_setup_multicast(ethip_nic_t *nic)
    371371{
    372372        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_nic_setup_multicast()");
    … …  
    425425        /* Setup the multicast MAC list */
    426426       
    427         int rc = nic_multicast_set_mode(nic->sess, NIC_MULTICAST_LIST,
     427        errno_t rc = nic_multicast_set_mode(nic->sess, NIC_MULTICAST_LIST,
    428428            mac_list, count);
    429429       
    … …  
    432432}
    433433
    434 int ethip_nic_addr_add(ethip_nic_t *nic, inet_addr_t *addr)
     434errno_t ethip_nic_addr_add(ethip_nic_t *nic, inet_addr_t *addr)
    435435{
    436436        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_nic_addr_add()");
    … …  
    445445}
    446446
    447 int ethip_nic_addr_remove(ethip_nic_t *nic, inet_addr_t *addr)
     447errno_t ethip_nic_addr_remove(ethip_nic_t *nic, inet_addr_t *addr)
    448448{
    449449        log_msg(LOG_DEFAULT, LVL_DEBUG, "ethip_nic_addr_remove()");

  • uspace/srv/net/ethip/ethip_nic.h

    r36f0738 rb7fd2a0  
    4242#include "ethip.h"
    4343
    44 extern int ethip_nic_discovery_start(void);
     44extern errno_t ethip_nic_discovery_start(void);
    4545extern ethip_nic_t *ethip_nic_find_by_iplink_sid(service_id_t);
    46 extern int ethip_nic_send(ethip_nic_t *, void *, size_t);
    47 extern int ethip_nic_addr_add(ethip_nic_t *, inet_addr_t *);
    48 extern int ethip_nic_addr_remove(ethip_nic_t *, inet_addr_t *);
     46extern errno_t ethip_nic_send(ethip_nic_t *, void *, size_t);
     47extern errno_t ethip_nic_addr_add(ethip_nic_t *, inet_addr_t *);
     48extern errno_t ethip_nic_addr_remove(ethip_nic_t *, inet_addr_t *);
    4949extern ethip_link_addr_t *ethip_nic_addr_find(ethip_nic_t *, inet_addr_t *);
    5050

  • uspace/srv/net/ethip/pdu.c

    r36f0738 rb7fd2a0  
    4747
    4848/** Encode Ethernet PDU. */
    49 int eth_pdu_encode(eth_frame_t *frame, void **rdata, size_t *rsize)
     49errno_t eth_pdu_encode(eth_frame_t *frame, void **rdata, size_t *rsize)
    5050{
    5151        void *data;
    … …  
    7575
    7676/** Decode Ethernet PDU. */
    77 int eth_pdu_decode(void *data, size_t size, eth_frame_t *frame)
     77errno_t eth_pdu_decode(void *data, size_t size, eth_frame_t *frame)
    7878{
    7979        eth_header_t *hdr;
    … …  
    106106
    107107/** Encode ARP PDU. */
    108 int arp_pdu_encode(arp_eth_packet_t *packet, void **rdata, size_t *rsize)
     108errno_t arp_pdu_encode(arp_eth_packet_t *packet, void **rdata, size_t *rsize)
    109109{
    110110        void *data;
    … …  
    149149
    150150/** Decode ARP PDU. */
    151 int arp_pdu_decode(void *data, size_t size, arp_eth_packet_t *packet)
     151errno_t arp_pdu_decode(void *data, size_t size, arp_eth_packet_t *packet)
    152152{
    153153        arp_eth_packet_fmt_t *pfmt;

  • uspace/srv/net/ethip/pdu.h

    r36f0738 rb7fd2a0  
    4040#include "ethip.h"
    4141
    42 extern int eth_pdu_encode(eth_frame_t *, void **, size_t *);
    43 extern int eth_pdu_decode(void *, size_t, eth_frame_t *);
    44 extern int arp_pdu_encode(arp_eth_packet_t *, void **, size_t *);
    45 extern int arp_pdu_decode(void *, size_t, arp_eth_packet_t *);
     42extern errno_t eth_pdu_encode(eth_frame_t *, void **, size_t *);
     43extern errno_t eth_pdu_decode(void *, size_t, eth_frame_t *);
     44extern errno_t arp_pdu_encode(arp_eth_packet_t *, void **, size_t *);
     45extern errno_t arp_pdu_decode(void *, size_t, arp_eth_packet_t *);
    4646
    4747#endif
Note: See TracChangeset for help on using the changeset viewer.