<p dir="ltr">On May 19, 2015 3:16 AM, "Dave Airlie" <<a href="mailto:airlied@gmail.com">airlied@gmail.com</a>> wrote:<br>
><br>
><br>
>>><br>
>>> drm-amdkfd-next-2015-05-18:<br>
>>><br>
>>> - Add the interrupts & events modules, including new IOCTLs to create and wait<br>
>>>   on events. The HSA RT open source stack is mainly using events to know when<br>
>>>   a dispatched work has been completed. In addition, this module is<br>
>>>   a pre-requisite for the next module I'm going to upstream - debugger support<br>
>>><br>
>>>   This module also handles H/W exceptions, such as memory exception received<br>
>>>   through the IOMMUv2 H/W and Bad Opcode exception receieved from the GPU.<br>
>>><br>
>>> - Adding a new kernel module parameter to let the user decide whether he wants<br>
>>>   to receive a SIGTERM when a memory exception occurs inside the GPU kernel and<br>
>>>   the HSA application doesn't wait on an appropriate event, or if he just want<br>
>>>   to receive notification about this event in dmesg. The default is the latter.<br>
>>><br>
>>> - Additional improvements for SDMA code<br>
>>><br>
>>> - Update my email address in Maintainers file as I'm leaving AMD in two weeks.<br>
><br>
><br>
> Two things,<br>
><br>
> a) pull request from ssh:// please fix scripts to generate git:// URLs,<br>
><br>
> b) ioctls with bool in them make me worry, four packed bool's even more,<br>
> 4 packed bools with a 64-bit straight after them put me over the not pulling this edge.<br>
><br>
> I don't trust bools in ioctl defintions, please use explicitly sized types.<br>
><br>
> Dave.</p>
<p dir="ltr">Thanks Dave, <br>
No problem, I'll fix and resend it today. </p>
<p dir="ltr">Oded</p>