| From 1a0a21f5619c2a649d53d2c54e38da7f796de690 Mon Sep 17 00:00:00 2001 |
| From: Philip A. Prindeville <philipp@redfish-solutions.com> |
| Date: Wed, 30 Mar 2011 12:59:26 +0000 |
| Subject: [PATCH] atm/solos-pci: Don't include frame pseudo-header on transmit |
| hex-dump |
| |
| commit 18b429e74eeafe42e947b1b0f9a760c7153a0b5c upstream. |
| |
| Omit pkt_hdr preamble when dumping transmitted packet as hex-dump; |
| we can pull this up because the frame has already been sent, and |
| dumping it is the last thing we do with it before freeing it. |
| |
| Also include the size, vpi, and vci in the debug as is done on |
| receive. |
| |
| Use "port" consistently instead of "device" intermittently. |
| |
| Signed-off-by: Philip Prindeville <philipp@redfish-solutions.com> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/drivers/atm/solos-pci.c b/drivers/atm/solos-pci.c |
| index 3613422..6ba851b 100644 |
| --- a/drivers/atm/solos-pci.c |
| +++ b/drivers/atm/solos-pci.c |
| @@ -695,7 +695,7 @@ void solos_bh(unsigned long card_arg) |
| size); |
| } |
| if (atmdebug) { |
| - dev_info(&card->dev->dev, "Received: device %d\n", port); |
| + dev_info(&card->dev->dev, "Received: port %d\n", port); |
| dev_info(&card->dev->dev, "size: %d VPI: %d VCI: %d\n", |
| size, le16_to_cpu(header->vpi), |
| le16_to_cpu(header->vci)); |
| @@ -1015,8 +1015,15 @@ static uint32_t fpga_tx(struct solos_card *card) |
| |
| /* Clean up and free oldskb now it's gone */ |
| if (atmdebug) { |
| + struct pkt_hdr *header = (void *)oldskb->data; |
| + int size = le16_to_cpu(header->size); |
| + |
| + skb_pull(oldskb, sizeof(*header)); |
| dev_info(&card->dev->dev, "Transmitted: port %d\n", |
| port); |
| + dev_info(&card->dev->dev, "size: %d VPI: %d VCI: %d\n", |
| + size, le16_to_cpu(header->vpi), |
| + le16_to_cpu(header->vci)); |
| print_buffer(oldskb); |
| } |
| |
| -- |
| 1.7.4.4 |
| |