linux-kernel - Re: [PATCH v9] can: usb: IXXAT USB-to-CAN adapters drivers

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]

Date:   Fri, 23 Jun 2023 14:09:32 +0200
From:   Peter Seiderer <ps.report@....net>
To:     Eric Stahl <ericstahl@...ntech.com>
Cc:     Vincent Mailhol <vincent.mailhol@...il.com>, marm@...-networks.de,
        linux-can@...r.kernel.org, Marc Kleine-Budde <mkl@...gutronix.de>,
        Wolfgang Grandegger <wg@...ndegger.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v9] can: usb: IXXAT USB-to-CAN adapters drivers

Hello Eric,

On Mon, 19 Jun 2023 10:25:27 -0400, Eric Stahl <ericstahl@...ntech.com> wrote:

> Hi all,
> 
> The socketcan@...-networks.de is bouncing for me. I removed both Florian 
> Ferg's email (flfe@...-networks.de) and sockectcan@...-networks.de. I 
> saw Markus from hms commenting on the last thread (PATCH v8) resolving 
> those issues, so I've added him to this chain. I'm not sure if they are 
> still a problem.
> 
> I've backported Peter's PATCH v9 and it works well for me. It seems that 

Thanks for testing effort ;-)

> hms is distributing a driver that does not incorporate the proposed 
> changes from the past reviews 
> (https://forum.hms-networks.com/t/socketcan-driver-for-linux-20-04/70299/30). 
> Their driver has not worked for me with higher level CANopen libraries, 

From a quick look at the provided sources they seem to (at least) lack the
following fix 'can: ixxat_usb: do not free skb before last usage'
(which was needed for my CAN FD use case to work with the HMS provided
sources):

	https://codeberg.org/psreport/socketcan-linux-ix-usb-can/commit/51b3021

> but Peter's v8/v9 and Florian's past contributions work well. I would 
> like to get ixxat's active pci socketcan drivers pulled into the mainline.
> 

Not on my agenda (as I have only access to USB-to-CAN FD hardware at the
moment)...

> I don't mean to hijack this thread, but I was wondering if it's 
> appropriate to add those pci drivers to this patch to get reviewed, or 
> if it's more appropriate to open a separate patch.

I think best is to provide it as a separate patch as a starting point...

Regards,
Peter
 
> 
> Thanks,
> 
> Eric