On Sun, Nov 01, 2020 at 12:31:05PM -0500, Arvind Sankar wrote: > Commit > b9de06783f01 ("compiler.h: fix barrier_data() on clang") > moved the definition of barrier() into compiler.h. That's not a real commit ID. It only exists in linux-next and will expire after a few weeks. The right way to fix a patch in Andrew's tree is to send an email asking him to apply it as a -fix patch. As part of Andrew's submission process, he folds all the -fix patches into the parent patch and it shows up pristine in Linus' tree. > This causes build failures at least on alpha, because there are files > that rely on barrier() being defined via the implicit include of > compiler_types.h. That seems like a bug that should be fixed rather than reverting this part of the patch?