← Back to context

Comment by gmokki

10 months ago

Do you mean location as in different subsystem than dma, which is required for all modern devices. Or just location as in stash the files outside the dma directory?

> Or just location as in stash the files outside the dma directory?

This one. The code in question, AFAIK, is a rust wrapper around the DMA subsystem. I think that does best belong in the DMA subsystem but with the maintainer unwilling to accept it the next place to put it is either in a new rust-dma subsystem/wrapper or to pull out the parts needed and integrate them directly in the drivers in question (certainly nasty and a lot of duplicate effort).

The goal for such code should ultimately be to cause as little friction as possible with the DMA maintainer.

If it comes up in the mailing list that "this is dumb and you shouldn't do this" that's when you loop in the DMA maintainer and request their suggestions on how to proceed with whoever challenges the change. Put them in the seat of making the decision of where the code should go.