<div dir="ltr"><br><br>On Wed, Jul 11, 2018 at 3:28 PM, Jan Stancek <<a href="mailto:jstancek@redhat.com">jstancek@redhat.com</a>> wrote:<br>><br>><br>> ----- Original Message -----<br>>> Maybe we misled by the code comments before, after thinking over, I<br>>> guess the "outside your accessible address space" != "outside of<br>>> process mapped space". So we don't need to do map/unmap to satisfy<br>>> that.<br>><br>> That may be the case, I changed it on purpose to match that comment.<br>> Though as you pointed out PROT_NONE might be enough too.<br>><br>> Is the motivation for your patch to re-use existing function<br><br>Yes, firstly my motivation is to re-use the existing function. But then I realized that maybe the original author misunderst<div class="gmail_default" style="font-size:small;display:inline">​oo​</div>d the EFAULT error in open().<br><br>So I find open(2) manual and get this:<br>"EFAULT pathname points outside your <b>accessible</b> address space."<div><br></div><div><div class="gmail_default" style="font-size:small">​Then I think maybe we need match the manual but not the code comments.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">That's all what I thought.​</div><br><br>> or have you hit some problem with map/unmap?<br><br>No, no other issues.<br><br><br>><br>> Regards,<br>> Jan<br><br><br><br>-- <br>Regards,<br>Li Wang</div></div>