netdev - Re: [PATCH 1/5] send-crc: Add framework to allow sending packets with customized CRC.

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

Message-ID: <4F318CF5.1060200@candelatech.com>
Date:	Tue, 07 Feb 2012 12:43:33 -0800
From:	Ben Greear <greearb@...delatech.com>
To:	Michał Mirosław <mirqus@...il.com>
CC:	netdev@...r.kernel.org
Subject: Re: [PATCH 1/5] send-crc: Add framework to allow sending packets
 with customized CRC.

On 02/07/2012 12:33 PM, Michał Mirosław wrote:
> 2012/2/7<greearb@...delatech.com>:
>> From: Ben Greear<greearb@...delatech.com>
>>
>> This is useful for testing RX handling of frames with bad
>> CRCs.
>>
>> Requires driver support to actually put the packet on the
>> wire properly.
> [...]
>
> I like this idea. I think that this is not a feature to have in
> dev->features to toggle, though. Driver either supports this or not,
> so it's better to put it in dev->priv_flags instead. Disabling of FCS
> inserting is a per-packet option in the drivers, isn't it?
>
> For one problem with toggling: if you create a socket using it and
> later turn off the feature, the socket will still send skbs with FCS
> appended.

For the priv-flags, is there a way to query that from user-space to
see if the NIC supports it?

The pkt-socket xmit logic will throw away pkts if the NIC doesn't
have the feature enabled, but a few already queued might get
through with extra CRC appended.

So, I'm fine with your suggestion, as long as there is some way to
query whether the NIC supports this feature or not...

Thanks,
Ben

>
> Best Regards,
> Michał Mirosław

-- 
Ben Greear <greearb@...delatech.com>
Candela Technologies Inc  http://www.candelatech.com

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