Le 18/09/2021 à 11:42, Arnd Bergmann a écrit :
On Sat, Sep 18, 2021 at 7:20 AM Christophe JAILLET
<christophe.jaillet@xxxxxxxxxx> wrote:
In [1], Christoph Hellwig has proposed to remove the wrappers in
include/linux/pci-dma-compat.h.
Some reasons why this API should be removed have been given by Julia
Lawall in [2].
Finally, Arnd Bergmann reminded that the documentation was updated 11 years
ago to only describe the modern linux/dma-mapping.h interfaces and mark the
old bus-specific ones as no longer recommended, see commit 216bf58f4092
("Documentation: convert PCI-DMA-mapping.txt to use the generic DMA API").
...
Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
Reviewed-by: Arnd Bergmann <arnd@xxxxxxxx>
How much is left now overall? Are you getting close enough that the
remaining patches to remove the header can be done as a short series and
merged through Andrew's -mm tree?
Arnd
Hi,
considering that the 3 patches that you have just review are done, the
diffsat of what is remaining is:
32 files changed, 309 insertions(+), 272 deletions(-)
All corresponding patches have already been proposed.
Most of the patches are not that old (i.e. < 1 month)
They all *look* trivial to me, *EXCEPT* the ones for message/fusion.
I've tried to split the latter in small series in the hope of easing review.
Moreover the drivers/message/fusion directory seems to be inactive since
a few months. I don't know if it is just related to the summer or if
this code is for some reason stalled.
This is funny, because the very start for me of this clean-up effort was
related to an issue in memory allocation in message/fusion fixed by
Christoph Hellwig :)
The details are:
Patches with a Review-by tag
============================
bus: mhi:
https://lore.kernel.org/kernel-janitors/bb3dc436fe142309a2334549db782c5ebb80a2be.1625718497.git.christophe.jaillet@xxxxxxxxxx/
fpga: dfl:
https://lore.kernel.org/kernel-janitors/c23cf1cfa058456da69849de22b957c6c414766d.1629643816.git.christophe.jaillet@xxxxxxxxxx/
Patches waiting for review
==========================
arch: sparc:
https://lore.kernel.org/kernel-janitors/aa28186920a1bb964ca03723e482f130cd8e9322.1629663425.git.christophe.jaillet@xxxxxxxxxx/
agp/intel:
https://lore.kernel.org/kernel-janitors/e6b5bc8d1f79955ebba652df3deff6b8b39cc607.1630136212.git.christophe.jaillet@xxxxxxxxxx/
i2c:
https://lore.kernel.org/kernel-janitors/fad542b558afc45496f7a7ba581593cd46e68f7c.1629660967.git.christophe.jaillet@xxxxxxxxxx/
mt76:
https://lore.kernel.org/kernel-janitors/83b2da6ff8a07d576fa3627051daa705aba37a3c.1629617782.git.christophe.jaillet@xxxxxxxxxx/
rapidio:
https://lore.kernel.org/kernel-janitors/20210117133929.563645-1-christophe.jaillet@xxxxxxxxxx/
--> Sent a long time ago, should be ping'ed
scsi:
https://lore.kernel.org/kernel-janitors/3899b1ed4abac581c30845d82f33ec6df8b38976.1629633207.git.christophe.jaillet@xxxxxxxxxx/
media: v4l2-pci-skeleton:
https://lore.kernel.org/kernel-janitors/7191e316ea3b61b6a015d063d2b068a9f4a965aa.1630230428.git.christophe.jaillet@xxxxxxxxxx/
--> Unsure if the one I sent the patch to is the correct one.
get_maintainer.pl is of no help here
message/fusion:
https://lore.kernel.org/kernel-janitors/cover.1623617903.git.christophe.jaillet@xxxxxxxxxx/
https://lore.kernel.org/kernel-janitors/db56a78d7d04b809abd32a6fb4839d698587bf7c.1623580326.git.christophe.jaillet@xxxxxxxxxx/
https://lore.kernel.org/kernel-janitors/cover.1623579808.git.christophe.jaillet@xxxxxxxxxx/
https://lore.kernel.org/kernel-janitors/cover.1623571676.git.christophe.jaillet@xxxxxxxxxx/
--> these series are a bit more tricky. See each cover letter
Patches that could be applied as-is, but which received comments
================================================================
drivers: dma:
https://lore.kernel.org/kernel-janitors/547fae4abef1ca3bf2198ca68e6c361b4d02f13c.1629635852.git.christophe.jaillet@xxxxxxxxxx/
--> Need to be rework because Andy Shevchenko (intel) wants some
additional clean-up.
Setting 64 bits DMA mask can't fail, so some code is dead and
should be removed.
I think that it could (should ?) be done in another step
drm/i915:
https://lore.kernel.org/kernel-janitors/dbf1018fb773785e0b3b40e601246ed6438e645e.1629666258.git.christophe.jaillet@xxxxxxxxxx/
--> The original code was apparently weird
--> I'm not sure on what do do. Send a v2 (which would be a blind
fix)? Send a serie? Wait for Robin Murphy to "do what needs to be done"?
I think that it could (should ?) be done in another step
Patches that I need to resend
=============================
arch: alpha:
https://lore.kernel.org/kernel-janitors/18ab2b83b05b50a00ef673e320c8ddc0d19f099a.1630230112.git.christophe.jaillet@xxxxxxxxxx/
--> subject is broken (s/sparc/alpha/)
--> I will send a v2 to fix it
Let me know what is the best way to finally end this clean-up which
started a *long* time ago. (see [1])
CJ
[1]:
https://lore.kernel.org/kernel-janitors/20200421081257.GA131897@xxxxxxxxxxxxx/
