dentproject / dentOS

dentOS SwitchDev based NOS
Other
200 stars 59 forks source link

net: prestera: Update prestera driver to v3.1.1 #157

Closed taraschornyiplv closed 2 years ago

taraschornyiplv commented 2 years ago

Switchdev Driver v3.1.1 includes:

Tested by Marvell Validation team on all based DNI/Accton platforms

Signed-off-by: Taras Chornyi taras.chornyi@plvision.eu

jmpolom commented 2 years ago

I would agree with the comments of @paulmenzel above…elaborating what was fixed (referencing back to issues) would be extremely helpful to end users. When we ask in the community meetings about the status of documentation for the Marvell driver, these PR messages are mentioned as ways to find out where things are at. However, when I have gone back through these messages it is not always clear what is being added or fixed in a given PR or commit. Ideally the content of these commit and PR messages should trace back directly to tracked issues somewhere.

Mickey201 commented 2 years ago

elaborating what was fixed (referencing back to issues) would be extremely helpful to end users.

John, The majority of this release isn't bug fixing. please refer to the commit comment. This is a new Switchdev driver release that includes a new PP support. Of course, there are several bug fixes in this release yet, none of them was raised by the repo 'issues'. The bottom line, moving forward with the in-tree kernel mode will make things more clear. Till then - new features or new PP support will be shared in the same way.

paulmenzel commented 2 years ago

Also:

Tested by Marvell Validation team on all based DNI/Accton platforms

What are “based DNI/Accton platforms”?

snobleaws commented 2 years ago

I would like to get this merged if possible so we can run the code scan and release dentOS v2. We will be moving to using a clone of the linux kernel directly in dentOS v3 so the patches will be separate for the kernel vs ONLP. See : https://github.com/dentproject/linux/tree/dent-linux-5.15.y

paulmenzel commented 2 years ago

I would like to get this merged if possible so we can run the code scan and release dentOS v2.

If providing a patch series is not possible, please ignore my comment.

We will be moving to using a clone of the linux kernel directly in dentOS v3 so the patches will be separate for the kernel vs ONLP. See : https://github.com/dentproject/linux/tree/dent-linux-5.15.y

Sounds great. Was there a public announcement or discussion?

sonoble commented 2 years ago

We will be moving to using a clone of the linux kernel directly in dentOS v3 so the patches will be separate for the kernel vs ONLP. See : https://github.com/dentproject/linux/tree/dent-linux-5.15.y

Sounds great. Was there a public announcement or discussion?

Yes it was discussed and decided on the TSC call in on December 15th. We are working on integrating it into the main image.

paulmenzel commented 2 years ago

I didn’t see a message with the minutes on the mailing list. Can I read the minutes somewhere?

snobleaws commented 2 years ago

I didn’t see a message with the minutes on the mailing list. Can I read the minutes somewhere?

https://docs.google.com/document/d/1NMOwvN7RF5m73ShYziNDJgt0Gk9ElQiY32day56Ik24