<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 11, 2018 at 3:54 PM, Jan Stancek <span dir="ltr"><<a href="mailto:jstancek@redhat.com" target="_blank">jstancek@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
----- Original Message -----<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<br>
> realized that maybe the original author misunderst<br>
> ​oo​<br>
> d the EFAULT error in open().<br>
> <br>
> So I find open(2) manual and get this:<br>
</span>> "EFAULT pathname points outside your *accessible* address space."<br>
<span class="">> <br>
> ​Then I think maybe we need match the manual but not the code comments.<br>
<br>
</span>I'd say we match it already (in stronger form than PROT_NONE).<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Right, <span style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">map/unmap is more strict.</span></div></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I don't object to re-using LTP function. Can you send v2<br>
that updates also the comment please?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">​Sure, though it's not much value to change the function, I will do that to make LTP happy.​</div></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Regards,<br>
Jan<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div>
</div></div>