Re: [PATCHv2] ARM: mvebu: use hardware I/O coherency also for PCI devices

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

 



On Tue, May 13, 2014 at 06:04:30PM +0200, Thomas Petazzoni wrote:
> Since the beginning of the introduction of hardware I/O coherency
> support for Armada 370 and Armada XP, the special DMA operations
> should have applied to all DMA capable devices. Unfortunately, while
> the original code properly took into account platform devices, it
> didn't take into account PCI devices, which can also be DMA masters.
> 
> This commit fixes that by registering a bus notifier on pci_bus_type,
> to register our custom DMA operations, like is already done for
> platform devices. While doing this, we also rename
> mvebu_hwcc_platform_notifier() to mvebu_hwcc_notifier() and
> mvebu_hwcc_platform_nb to mvebu_hwcc_nb because they are no longer
> specific to platform devices.
> 
> Fixes: e60304f8cb7b ("arm: mvebu: Add hardware I/O Coherency support")
> Cc: <stable@xxxxxxxxxxxxxxx> # v3.8+
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxxxxxxxxx>
> ---
> Changes since v1:
>  * Fix stupid tab vs. space problem
> 
> Applies to mvebu/soc.
> 
> Whether this commit qualifies as stable material can be
> discussed. Technically, it was a mistake from the original I/O
> coherency implementation. But in practice, besides not using the I/O
> coherency, I don't think it's causing any specific problem other than
> DMA transactions on PCI devices triggering unneeded cache maintenance
> operations. So I'll leave it to the mvebu maintainers to decide
> whether this qualifies as stable material or not. I've included the
> relevant Fixes: and Cc: fields, but don't hesitate to rip them out if
> you don't think this should go to stable.

If nothing is broken for users, I see no point backporting it.  Feel
free to convince me otherwise...

thx,

Jason.
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]