lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <f143b58d-4caa-7c9b-b98b-806ba8d2be99@gmail.com>
Date:   Wed, 15 Apr 2020 18:18:15 +0200
From:   Heiner Kallweit <hkallweit1@...il.com>
To:     Lauri Jakku <ljakku77@...il.com>
Cc:     Leon Romanovsky <leon@...nel.org>, netdev@...r.kernel.org,
        nic_swsd@...ltek.com
Subject: Re: NET: r8168/r8169 identifying fix

On 15.04.2020 16:39, Lauri Jakku wrote:
> Hi,
> 
> There seems to he Something odd problem, maybe timing related. Stripped version not workingas expected. I get back to you, when  i have it working.

There's no point in working on your patch. W/o proper justification it
isn't acceptable anyway. And so far we still don't know which problem
you actually have.
FIRST please provide the requested logs and explain the actual problem
(incl. the commit that caused the regression).


> 13. huhtik. 2020, 14.46, Lauri Jakku <ljakku77@...il.com <mailto:ljakku77@...il.com>> kirjoitti:
> 
>     Hi,
> 
>     Fair enough, i'll strip them.
> 
>     -lja
> 
>     On 2020-04-13 14:34, Leon Romanovsky wrote:
> 
>         On Mon, Apr 13, 2020 at 02:02:01PM +0300, Lauri Jakku wrote:
> 
>             Hi,
> 
>             Comments inline.
> 
>             On 2020-04-13 13:58, Leon Romanovsky wrote:
> 
>                 On Mon, Apr 13, 2020 at 01:30:13PM +0300, Lauri Jakku wrote:
> 
>                     From 2d41edd4e6455187094f3a13d58c46eeee35aa31 Mon Sep 17 00:00:00 2001
>                     From: Lauri Jakku <lja@....fi>
>                     Date: Mon, 13 Apr 2020 13:18:35 +0300
>                     Subject: [PATCH] NET: r8168/r8169 identifying fix
> 
>                     The driver installation determination made properly by
>                     checking PHY vs DRIVER id's.
>                     ---
>                     drivers/net/ethernet/realtek/r8169_main.c | 70 ++++++++++++++++++++---
>                     drivers/net/phy/mdio_bus.c | 11 +++-
>                     2 files changed, 72 insertions(+), 9 deletions(-)
> 
> 
>                 I would say that most of the code is debug prints.
> 
> 
> 
>             I tought that they are helpful to keep, they are using the debug calls, so
>             they are not visible if user does not like those.
> 
> 
>         You are missing the point of who are your users.
> 
>         Users want to have working device and the code. They don't need or like
>         to debug their kernel.
> 
>         Thanks
> 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ