Dear Benjamin Herrenschmidt, On Thu, 08 Aug 2013 08:45:36 +1000, Benjamin Herrenschmidt wrote: > > > - You don't need my ack since I am not the maintainer of the irqdomain > > > code anymore, Grant is :-) > > > > Hrm, I'm being told Grant isn't anymore... I can step in and take it all > > back but you might not like the result .... > > Oh but it looks like MAINTAINERS says I am :-) > > Ok so from that perspective, I don't like it at all. You can try to > convince me otherwise but I don't think we need to introduce a > dependency to something like msi into the core remapper. It's already to > complex. > > Why don't you move the powerpc bitmap allocator over to a generic > place ? I feel like it would be actually simpler but feel free to prove > me wrong. I'm sorry, but I'm not buying this. There must be some continuity when the maintenance of one subsystem transitions from one maintainer to another. I'm perfectly ok with accepting some hick-ups, but not radical changes in design decisions. What you're asking me to do is to go completely backwards compared to the comments and review Grant made. The irqdomain-based allocator was suggested by Grant (see my previous e-mail, or Grant reply at http://lists.infradead.org/pipermail/linux-arm-kernel/2013-June/175430.html) and was even Acked-by Grant in http://lists.infradead.org/pipermail/linux-arm-kernel/2013-July/187082.html. Note that this patch set has been posted at the following dates: * PATCH version 7 sent on August, 7th 2013 * PATCH version 6 sent on August, 1st 2013 * PATCH version 5 sent on July, 15th 2013 * PATCH version 4 sent on July, 1st 2013 * PATCH version 3 sent on June, 19th 2013 * PATCH version 2 sent on June, 6th 2013 * RFC version 1 sent on March, 26th 2013 So it has been around since 4 months, I've taken into account all the comments from the various maintainers who were involved, and especially the comments from Grant. You cannot ask me now, as we are approaching the next merge window for which this code is intended, to take completely opposite design choices than what the previous irqdomain maintainer was suggesting. Could you contact Grant and align with him on those design decisions? It would also be good if you could read the past discussions on this patch set, because all what you're pointing at has already been discussed at length, as I pointed out in my previous e-mail. Thanks, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html