Hi Leon - the code to which this patch applies handles a data stream from our card's firmware to the host (over PCI). There are device-specific registers which are accessed to transfer log data from our firmware to the host. I don't think there is kernel code that we could use to perform this; this is not general purpose host driver logging.
Rick -----Original Message----- From: Leon Romanovsky [mailto:[email protected]] Sent: Saturday, August 12, 2017 10:56 PM To: Manlunas, Felix <[email protected]> Cc: [email protected]; [email protected]; Vatsavayi, Raghu <[email protected]>; Chickles, Derek <[email protected]>; Burla, Satananda <[email protected]>; Ricardo Farrington <[email protected]> Subject: Re: [PATCH net-next] liquidio: update debug console logging mechanism On Fri, Aug 11, 2017 at 06:43:14PM -0700, Felix Manlunas wrote: > From: Rick Farrington <[email protected]> > > - remove logging dependency upon global func > octeon_console_debug_enabled() > - abstract debug console logging using console structure (via function ptr) > to allow for more flexible logging > > Signed-off-by: Rick Farrington <[email protected]> > Signed-off-by: Raghu Vatsavayi <[email protected]> > Signed-off-by: Felix Manlunas <[email protected]> > --- > drivers/net/ethernet/cavium/liquidio/lio_main.c | 44 +++++++++++++++++- > .../net/ethernet/cavium/liquidio/octeon_console.c | 54 > ++++++++++++++-------- > .../net/ethernet/cavium/liquidio/octeon_device.h | 17 +++++-- > 3 files changed, 90 insertions(+), 25 deletions(-) > I'm probably missing something important, but why do you need your custom console implementation if kernel is full of such built-in options? Thanks
