03.09.2013 12:35, Andreas Färber wrote: > I also don't understand why qemu-trivial is suddenly picking up Stefan's > arm translation patch, it used to be for unmaintained areas only. But > arm is not my problem.
Which patch you're talking about? Is it "target-arm: Report unimplemented opcodes (LOG_UNIMP)" ? If yes, that one appears to be trivial as it just adds some logging before failing an instruction and should not conflict with other work being done in this area. Perhaps I was too aggressive while picking up the backlog. We should just draw the line *somewhere*, -- eg, it sure is possible to reject spelling fixes for maintained areas from -trivial (like this arm tree), - will this be productive? This change (cputlb: remove dead function) appears to be "trivial enough" for me (after looking at the usage history of this function), and I'd pick it up without this Andreas's request, too. As for the "suddenly" - it's not really suddenly, it's because it has been Cc'd to -trivial (by someone who submitted lots of good trivial patches before) and actually looks trivial, too. And also because subsystem maintainer added his Reviewed-by, apparently (or hopefully) after noticing it's submitted to -trivial. I also Cc'd both maintainers in my notice that it's been applied to -trivial. Speaking of linux headers sync, I did that once indeed, but don't think it was a good idea. It is "trivial" in a sense that it just makes headers in qemu to be the same as in current kernel (this is easy to verify), and the tree - at least in some configuration - compiles. But indeed, the side effects might be quite a bit unexpected and "non-trivial" - in other words, it is a "trivial change with non-trivial possible consequences". HTH. /mjt