<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Apr 15, 2016 at 6:58 AM Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Apr 14, 2016 at 1:56 PM Reid Kleckner <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">I think most function redirection patching schemes are going to be mutually incompatible, so I'm not sure it makes sense to make this attribute a comma-separated list.<br></div><div class="gmail_quote"><br></div></div></div></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>Isn't this what I said? :)</div></div></div><div dir="ltr"><div class="gmail_quote"><div></div></div></div></blockquote><div><br></div><div>While some schemes may be incompatible, I suspect composing some of them together makes some sense. For example:</div><div><br></div><div>patchable-function=hotpatch-short-prologue,hotpatch-short-epilogue</div><div><br></div><div>We can catch some incompatibilities in the implementation if there are serious problems with mixing them.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"></div><div class="gmail_quote">I think Eric's and Dean's use case may be better addressed by a separate attribute. My recollection is that they want to add nop slides to the prologue and epilogue that can be hotpatched to enable and disable instrumentation at runtime.</div></div></div></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>I'm not sure about this, it'd be nice not to have a bajillion more attributes.</div></div></div><div dir="ltr"><div class="gmail_quote"><div><br></div></div></div></blockquote><div><br></div><div>I for one would like less attributes. It seems this is potentially something we can use for a broader purpose so I'd like to explore that possibility early (rather than doing it later).</div><div><br></div><div>Cheers</div></div></div>