mbox series

[net-next,0/2] nfp: fix rtsym and XDP register handling in debug dump

Message ID 20171214095026.11665-1-simon.horman@netronome.com
Headers show
Series nfp: fix rtsym and XDP register handling in debug dump | expand

Message

Simon Horman Dec. 14, 2017, 9:50 a.m. UTC
Hi,

this series resolves two problems in the recently added debug dump facility.

* Correctly handle reading absolute rtysms
* Correctly handle special-case PB register reads

These fixes are for code only present in net-next.

Carl Heymann (2):
  nfp: fix absolute rtsym handling in debug dump
  nfp: fix XPB register reads in debug dump

 .../net/ethernet/netronome/nfp/nfp_net_debugdump.c | 57 +++++++++++++++-------
 1 file changed, 39 insertions(+), 18 deletions(-)

Comments

Simon Horman Dec. 15, 2017, 2:18 p.m. UTC | #1
On Thu, Dec 14, 2017 at 10:50:24AM +0100, Simon Horman wrote:
> Hi,
> 
> this series resolves two problems in the recently added debug dump facility.
> 
> * Correctly handle reading absolute rtysms
> * Correctly handle special-case PB register reads
> 
> These fixes are for code only present in net-next.

Hi Dave,

It seems that I made a thinko in the title of the cover letter.  s/XDP/XPB/
These changes do not relate to XDP.

> Carl Heymann (2):
>   nfp: fix absolute rtsym handling in debug dump
>   nfp: fix XPB register reads in debug dump
> 
>  .../net/ethernet/netronome/nfp/nfp_net_debugdump.c | 57 +++++++++++++++-------
>  1 file changed, 39 insertions(+), 18 deletions(-)
> 
> -- 
> 2.11.0
>
David Miller Dec. 15, 2017, 5:49 p.m. UTC | #2
From: Simon Horman <simon.horman@netronome.com>
Date: Fri, 15 Dec 2017 15:18:29 +0100

> On Thu, Dec 14, 2017 at 10:50:24AM +0100, Simon Horman wrote:
>> Hi,
>> 
>> this series resolves two problems in the recently added debug dump facility.
>> 
>> * Correctly handle reading absolute rtysms
>> * Correctly handle special-case PB register reads
>> 
>> These fixes are for code only present in net-next.
> 
> Hi Dave,
> 
> It seems that I made a thinko in the title of the cover letter.  s/XDP/XPB/
> These changes do not relate to XDP.

Series applied with this fixed, thanks.