<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - [Patch?] DisplayPort MST (Multistream Transport) hotplug kernel memory fault" href="https://bugs.freedesktop.org/show_bug.cgi?id=91481#c3">Comment # 3</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - [Patch?] DisplayPort MST (Multistream Transport) hotplug kernel memory fault" href="https://bugs.freedesktop.org/show_bug.cgi?id=91481">bug 91481</a> from <a class="email" href="mailto:adam_richter2004@yahoo.com" title="Adam J. Richter <adam_richter2004@yahoo.com>"> Adam J. Richter</a> <pre>Hi, Dave. Thank you for posting your patch to the dri-devel mailing list (at <a href="http://article.gmane.org/gmane.comp.video.dri.devel/136173">http://article.gmane.org/gmane.comp.video.dri.devel/136173</a> ). Your patch is probably fine in practice, although I am not sure it deals correctly with the the ways of drm_dp_send_link_address can fail. Part of the reason I made my patch the way I did was that I guessed that if the kzalloc at the beginning of drm_dp_send_link_address failed (which probably will never happen), that it might be better not to set link_address_sent to true. On the other hand, I was not sure what to do with the "link address failed" and "link address nak received" cases in that function. Perhaps link_address_sent should not be set in those cases either. Anyhow, pushing your patch upstream would certainly be an improvement. I'm all for it if you think you patch is actually correct in these cases too, or if you want to push your patch upstream and think about corner cases later. If you do have any thoughts on these corner cases, your comments would be most welcome of course. Thanks again for addressing this bug.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>