Thanks Will for taking a look. >> The device memory such as on the Grace Hopper systems is interchangeable >> with DDR memory and retains its properties. Allow executable faults >> on the memory determined as Normal cacheable. > > Sorry, but a change this subtle to the arch code is going to need a _much_ > better explanation than the rambling text above. Understood, I'll work on the text and try to make it coherent. > I also suspect that you're trying to do too many things in one patch. > In particular, the changes to execute permission handling absolutely > need to be discussed as a single patch in the series. > > Please can you split this up in to a set of changes with useful commit > messages so that we can review them? Yes. I'll take out the execute permission part to a separate patch. > Jason knows how to do this so you could ask him for help if you're stuck. > Otherwise, there are plenty of examples of well-written commit messages > on the mailing list and in the git history. Ack.