# Re: \[PATCH\] powerpc/mm: use \_raw variant of page table accessors

- From: Balbir Singh \<bsingharora@gmail.com\>
- Date: 2016-06-06 05:33:55 
- List: linuxppc-dev
- Page: https://ratatoskr.run/linuxppc-dev/2016/06/9490315
- Thread: https://ratatoskr.run/linuxppc-dev/2016/05/9490196/t.md

```text


On 06/06/16 08:30, Benjamin Herrenschmidt wrote:
> On Fri, 2016-06-03 at 15:34 +1000, Balbir Singh wrote:
>>
>> Can we just save the cpu_to_be64(_PAGE_PTE) as _BE64_PAGE_PTE constant
>> in big-endian (pgtable-be-types.h) and similar for other things. I know its
>> not the best option, but we don't really expect these bits to change often or anymore
>>
> How does that help ? The compiler solves it at compile time either
> way, I'm not sure adding yet another constant will improve
> readability here.
> 

I think your right, but having those constants will prevent people from converting from
be64 to cpu, then or'ing and converting back. As long as programmers stick to pxx_raw()
for manipulation we should be in good shape. I was hoping the BE constants would do that,
but its just a matter of discipline

Balbir
```