Got rid of an unnecessary memcpy() in the network queue code.
This commit is contained in:
@@ -240,16 +240,12 @@ network_queue_put(void *priv, uint8_t *data, int len)
|
|||||||
|
|
||||||
|
|
||||||
static void
|
static void
|
||||||
network_queue_get(netpkt_t *pkt)
|
network_queue_get(netpkt_t **pkt)
|
||||||
{
|
{
|
||||||
pkt->priv = NULL;
|
|
||||||
pkt->data = NULL;
|
|
||||||
pkt->len = 0;
|
|
||||||
|
|
||||||
if (first_pkt == NULL)
|
if (first_pkt == NULL)
|
||||||
return;
|
*pkt = NULL;
|
||||||
|
else
|
||||||
memcpy(pkt, first_pkt, sizeof(netpkt_t));
|
*pkt = first_pkt;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
@@ -295,15 +291,15 @@ network_queue_clear(void)
|
|||||||
static void
|
static void
|
||||||
network_queue(void *priv)
|
network_queue(void *priv)
|
||||||
{
|
{
|
||||||
netpkt_t pkt;
|
netpkt_t *pkt = NULL;
|
||||||
|
|
||||||
network_busy(1);
|
network_busy(1);
|
||||||
|
|
||||||
network_queue_get(&pkt);
|
network_queue_get(&pkt);
|
||||||
if (pkt.len > 0) {
|
if ((pkt != NULL) && (pkt->len > 0)) {
|
||||||
net_cards[network_card].rx(pkt.priv, pkt.data, pkt.len);
|
net_cards[network_card].rx(pkt->priv, pkt->data, pkt->len);
|
||||||
if (pkt.len >= 128)
|
if (pkt->len >= 128)
|
||||||
timer_on_auto(&network_queue_timer, 0.762939453125 * 2.0 * ((double) pkt.len));
|
timer_on_auto(&network_queue_timer, 0.762939453125 * 2.0 * ((double) pkt->len));
|
||||||
else
|
else
|
||||||
timer_on_auto(&network_queue_timer, 0.762939453125 * 2.0 * 128.0);
|
timer_on_auto(&network_queue_timer, 0.762939453125 * 2.0 * 128.0);
|
||||||
} else
|
} else
|
||||||
|
|||||||
Reference in New Issue
Block a user