On Mon, Sep 26, 2011 at 5:48 PM, KyongHo Cho <pullip.cho@xxxxxxxxxxx> wrote: > I am sorry but I still think that > installing default fault handler is quite simple and straightforward. ... and abusing the IOMMU API. Please don't do that. Interfaces are written for specific goals, and this one was added for IOMMU users, not drivers. Just like all the other map/unmap/attach/... APIs. By using this from the IOMMU driver itself, you are adding burden on future development and evolution of this API. As use cases materialize, we will have to change it to support them, while considering existing use cases. Using this API inappropriately will make our life harder later on. Moreover, I'm really not sure how exactly are you going to use this. If you're having an IOMMU user which installs its own fault handler, then this is all moot since the default behavior you provide here will be overridden. If you're not having an IOMMU user which installs its own fault handler, then just provide the default behavior in your irq handler, regardless whether you report the fault to the IOMMU core or not. Or only when it fails. and yes, we can provide different error values for different scenarios. it's not rocket science :). Really, there's a myriad of ways to do this right. Please explain your exact settings/use-case and I'll gladly help you find a clean solution. Thanks, Ohad. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |