packet: fix a couple of cppcheck warnings

Doesn't bring much, but also doesn't hurt us to fix 'em:

1) In tpacket_rcv() flush dcache page we can restirct the scope
   for start and end and remove one layer of indent.

2) In tpacket_destruct_skb() we can restirct the scope for ph.

3) In alloc_one_pg_vec_page() we can remove the NULL assignment
   and change spacing a bit.

Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Daniel Borkmann 2014-01-19 11:46:53 +01:00 committed by David S. Miller
parent 967680e027
commit f0d4eb29d1
1 changed files with 15 additions and 22 deletions

View File

@ -2009,19 +2009,20 @@ static int tpacket_rcv(struct sk_buff *skb, struct net_device *dev,
sll->sll_ifindex = dev->ifindex; sll->sll_ifindex = dev->ifindex;
smp_mb(); smp_mb();
#if ARCH_IMPLEMENTS_FLUSH_DCACHE_PAGE == 1 #if ARCH_IMPLEMENTS_FLUSH_DCACHE_PAGE == 1
{ if (po->tp_version <= TPACKET_V2) {
u8 *start, *end; u8 *start, *end;
if (po->tp_version <= TPACKET_V2) { end = (u8 *) PAGE_ALIGN((unsigned long) h.raw +
end = (u8 *)PAGE_ALIGN((unsigned long)h.raw macoff + snaplen);
+ macoff + snaplen);
for (start = h.raw; start < end; start += PAGE_SIZE) for (start = h.raw; start < end; start += PAGE_SIZE)
flush_dcache_page(pgv_to_page(start)); flush_dcache_page(pgv_to_page(start));
} }
smp_wmb(); smp_wmb();
}
#endif #endif
if (po->tp_version <= TPACKET_V2) if (po->tp_version <= TPACKET_V2)
__packet_set_status(po, h.raw, status); __packet_set_status(po, h.raw, status);
else else
@ -2050,9 +2051,9 @@ ring_is_full:
static void tpacket_destruct_skb(struct sk_buff *skb) static void tpacket_destruct_skb(struct sk_buff *skb)
{ {
struct packet_sock *po = pkt_sk(skb->sk); struct packet_sock *po = pkt_sk(skb->sk);
void *ph;
if (likely(po->tx_ring.pg_vec)) { if (likely(po->tx_ring.pg_vec)) {
void *ph;
__u32 ts; __u32 ts;
ph = skb_shinfo(skb)->destructor_arg; ph = skb_shinfo(skb)->destructor_arg;
@ -3649,34 +3650,26 @@ static void free_pg_vec(struct pgv *pg_vec, unsigned int order,
static char *alloc_one_pg_vec_page(unsigned long order) static char *alloc_one_pg_vec_page(unsigned long order)
{ {
char *buffer = NULL; char *buffer;
gfp_t gfp_flags = GFP_KERNEL | __GFP_COMP | gfp_t gfp_flags = GFP_KERNEL | __GFP_COMP |
__GFP_ZERO | __GFP_NOWARN | __GFP_NORETRY; __GFP_ZERO | __GFP_NOWARN | __GFP_NORETRY;
buffer = (char *) __get_free_pages(gfp_flags, order); buffer = (char *) __get_free_pages(gfp_flags, order);
if (buffer) if (buffer)
return buffer; return buffer;
/* /* __get_free_pages failed, fall back to vmalloc */
* __get_free_pages failed, fall back to vmalloc
*/
buffer = vzalloc((1 << order) * PAGE_SIZE); buffer = vzalloc((1 << order) * PAGE_SIZE);
if (buffer) if (buffer)
return buffer; return buffer;
/* /* vmalloc failed, lets dig into swap here */
* vmalloc failed, lets dig into swap here
*/
gfp_flags &= ~__GFP_NORETRY; gfp_flags &= ~__GFP_NORETRY;
buffer = (char *)__get_free_pages(gfp_flags, order); buffer = (char *) __get_free_pages(gfp_flags, order);
if (buffer) if (buffer)
return buffer; return buffer;
/* /* complete and utter failure */
* complete and utter failure
*/
return NULL; return NULL;
} }