On Sat, Oct 10, 2020 at 10:40:55PM +0900, Benjamin Poirier wrote:
On 2020-10-10 18:00 +0800, Coiby Xu wrote:
[...]
>
> Please also update drivers/staging/qlge/TODO accordingly. There is still
> a lot of debugging code IMO (the netif_printk statements - kernel
> tracing can be used instead of
On 2020-10-10 18:00 +0800, Coiby Xu wrote:
[...]
> >
> > Please also update drivers/staging/qlge/TODO accordingly. There is still
> > a lot of debugging code IMO (the netif_printk statements - kernel
> > tracing can be used instead of those) but this patch is a substantial
> > improvement.
>
>
On Sat, Oct 10, 2020 at 05:01:26PM +0900, Benjamin Poirier wrote:
On 2020-10-08 19:58 +0800, Coiby Xu wrote:
The debugging code in the following ifdef land
- QL_ALL_DUMP
- QL_REG_DUMP
- QL_DEV_DUMP
- QL_CB_DUMP
- QL_IB_DUMP
- QL_OB_DUMP
becomes unnecessary because,
- Device status and
On 2020-10-08 19:58 +0800, Coiby Xu wrote:
> The debugging code in the following ifdef land
> - QL_ALL_DUMP
> - QL_REG_DUMP
> - QL_DEV_DUMP
> - QL_CB_DUMP
> - QL_IB_DUMP
> - QL_OB_DUMP
>
> becomes unnecessary because,
> - Device status and general registers can be obtained by ethtool.
> -
The debugging code in the following ifdef land
- QL_ALL_DUMP
- QL_REG_DUMP
- QL_DEV_DUMP
- QL_CB_DUMP
- QL_IB_DUMP
- QL_OB_DUMP
becomes unnecessary because,
- Device status and general registers can be obtained by ethtool.
- Coredump can be done via devlink health reporter.
- Structure
5 matches
Mail list logo