On Mon, Aug 13, 2018 at 9:14 AM Matthew Wilcox <willy@xxxxxxxxxxxxx> wrote: > > Please consider pulling the XArray patch set. So this merge window has been horrible, but I was just about to start looking at it. And no. I'm not going to pull this. For some unfathomable reason, you have based it on the libnvdimm tree. I don't understand at all wjhy you did that. That libnvdimm tree didn't get merged., because it had complete garbage in the mm/ code. And yes, that buggy shit was what you based the radix tree code on. I seriously have no idea why you have based it on some unstable random tree in the first place. But basing it on something that I independently refused to pull because of obvious bugs from just a quick scan - that completely invalidates this pull request. Why? I guess it makes this merge window easier, since now I don't even have to look at the code, but it annoys the hell out of me when things like that happen. There wasn't even a mention in the pull request about how this was all based on some libnvdimm code that hadn't been merged yet. But you must have known that, since you must have explicitly done the pull request not against my tree, but against the bogus base branch. And since I won't be merging this, I clearly won't be merging your other pull request that depended on this either. Why the f*ck were these features so interlinked to begin with? Linus