netdev - Re: issues with commit usbnet: mcs7830: apply usbnet_link

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-prev] [day] [month] [year] [list]

Message-ID: <CACVXFVN3z8Hd4W_T+7XLvjFb4RmskzgHCAeoGsBvOeVPHE-J4A@mail.gmail.com>
Date:	Wed, 31 Jul 2013 23:57:12 +0800
From:	Ming Lei <ming.lei@...onical.com>
To:	Till Schmalmack <till@...malmack.net>
Cc:	Network Development <netdev@...r.kernel.org>,
	linux-usb <linux-usb@...r.kernel.org>
Subject: Re: issues with commit usbnet: mcs7830: apply usbnet_link_change

On Wed, Jul 31, 2013 at 11:49 PM, Ming Lei <ming.lei@...onical.com> wrote:
> On Wed, Jul 31, 2013 at 5:56 PM, Till Schmalmack <till@...malmack.net> wrote:
>> Hello Ming,
>>
>> I am experiencing issues with the usbnet commits you did for the 3.10
>> kernel, in particular with
>>
>> 4be74c13 usbnet: mcs7830: apply usbnet_link_change
>>
>> I filed this bug
>>
>> https://bugzilla.kernel.org/show_bug.cgi?id=60662
>
> Thanks for the report.
>
> Looks you didn't provide any debug info, could you apply the attachment
> patch and post the 'dmesg' info when you reproduce the bug.

Also I am wondering the commit 4be74c13(usbnet: mcs7830: apply
usbnet_link_change) may cause the regression.

If you look at commit 4be74c13(usbnet: mcs7830: apply usbnet_link_change)
and commit ac64995da(usbnet: introduce usbnet_link_change API),
you may find basically no real code changes on msc7830 drivers, and only
the below code is moved into usbnet_link_change() from mcs7830_status, isn't it?

       if (link)
               netif_carrier_on(dev->net);
       else
               netif_carrier_off(dev->net);



Thanks,
--
Ming Lei
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html