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  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]
Date:	Wed, 05 Mar 2014 11:56:16 +0100
From:	Ivan Vecera <ivecera@...hat.com>
To:	Sergei Shtylyov <sergei.shtylyov@...entembedded.com>,
	netdev@...r.kernel.org
CC:	sathya.perla@...lex.com, subbu.seetharaman@...lex.com,
	ajit.khaparde@...lex.com
Subject: Re: [PATCH net-next] be2net: do external loopback test only when
 it is requested

On 03/04/2014 09:17 PM, Sergei Shtylyov wrote:
> Hello.
>
> On 03/04/2014 09:04 PM, Ivan Vecera wrote:
>
>> Cc: sathya.perla@...lex.com
>> Cc: subbu.seetharaman@...lex.com
>> Cc: ajit.khaparde@...lex.com
>
>> Signed-off-by: Ivan Vecera <ivecera@...hat.com>
>> ---
>>   drivers/net/ethernet/emulex/benet/be_ethtool.c | 9 ++++++---
>>   1 file changed, 6 insertions(+), 3 deletions(-)
>
>> diff --git a/drivers/net/ethernet/emulex/benet/be_ethtool.c
>> b/drivers/net/ethernet/emulex/benet/be_ethtool.c
>> index cf09d8f..957848f 100644
>> --- a/drivers/net/ethernet/emulex/benet/be_ethtool.c
>> +++ b/drivers/net/ethernet/emulex/benet/be_ethtool.c
>> @@ -809,9 +809,12 @@ be_self_test(struct net_device *netdev, struct
>> ethtool_test *test, u64 *data)
>>                           &data[1]) != 0) {
>>               test->flags |= ETH_TEST_FL_FAILED;
>>           }
>> -        if (be_loopback_test(adapter, BE_ONE_PORT_EXT_LOOPBACK,
>> -                        &data[2]) != 0) {
>> -            test->flags |= ETH_TEST_FL_FAILED;
>> +        if (test->flags & ETH_TEST_FL_EXTERNAL_LB) {
>> +            if (be_loopback_test(adapter, BE_ONE_PORT_EXT_LOOPBACK,
>> +                         &data[2]) != 0) {
>> +                test->flags |= ETH_TEST_FL_FAILED;
>> +            }
>
>     {} not needed here, according to Documentation/CodingStyle. I'm
> surprised scripts/checkpatch.pl passed, it used to catch those.
>
I wanted to preserve the same look as for other prior loopback 
if-blocks. I will fix them all and resend v2.

Ivan

--
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

Powered by blists - more mailing lists