Re: [PATCH] PCI: mvebu: Mark driver as BROKEN

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 08 August 2023 11:26:27 Bjorn Helgaas wrote:
> [+cc linux-arm-kernel, beginning of thread:
> https://lore.kernel.org/r/20230114164125.1298-1-pali@xxxxxxxxxx]
> 
> On Tue, Aug 08, 2023 at 09:26:05AM +0200, Pali Rohár wrote:
> > On Friday 04 August 2023 12:35:13 Russell King (Oracle) wrote:
> > ...
> 
> > > For example, I have an Atheros PCIe WiFi card in an Armada 388 Clearfog
> > > platform, and this works fine.
> > > 
> > > Uwe has a SATA controller for a bunch of disks in an Armada 370 based
> > > NAS platform that is connected to PCIe, and removing PCIe support
> > > effectively makes his platform utterly useless.
> > > 
> > > Please revert this patch.
> > 
> > Please do not revert it, instead start fixing problems.
> 
> We know that like all the other drivers, the mvebu driver isn't
> perfect.  But I don't think effectively removing the driver completely
> helps anybody.  If people try to use it and notice problems, we have a
> chance to try to fix them.

I do not want to remove it. I was trying to find somebody who can start
caring about issues. In last year I was resending patches, some smaller
which could improve situation, but most of them were ignored or rejected.

So I'm here and waiting for alternatives, and I'm prepared to review
changes and patches for mvebu, which can improve driver support.

But I do not see anything. The only one who wrote something useful was
Uwe as he wanted to do some git bisect (which normally indicates issues
or also fixup/patch).

Also some times ago Greg wrote something like that (mainline) kernel is
place for unsupported and broken drivers. But mvebu is going in this
direction.

How can I otherwise point out to start doing something in this area?

Or are you unhappy with the fact that there is at least somebody (me)
who is willing to do patch review for this marvell stuff? You should
have said it to me earlier.

But as I'm reading now, that I should go away, maybe you should have to
find also new reviewer for driver. Good luck here as there was nobody
who even wanted to do anything in this area.

> Or maybe I'm missing your point.  I think you're suggesting that we
> keep pci-mvebu in the tree but unselectable because it depends on
> CONFIG_BROKEN.  What would be the advantage of doing that?
> 
> Bjorn

Well, all knows here that driver is in bad state. In past there were
regressions and no accepted fixes for it. (At that time I prepared fix,
but you did not like it and nobody else comes with other alternative
patch).

There area other options which can be done now, if there are only people
like Russel who are complaining but refusing to do absolutely nothing.
For example mark driver as experimental (there is some Kconfig symbol
for it). Or add a new menuconfig selectable symbol which appropriately
warn all distributions about problems and would be dependency for mvebu.
(if you do not like broken symbol).



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux