This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v3 5/5] Add support for Intel PKRU register to GDB and GDBserver.


On 01/27/2017 03:00 PM, Sturm, Michael wrote:

>> In the test:
>>
>>> +#include <stdio.h>
>> This is not needed, right?
> It's needed to pull in the definition for NULL which is used in the call
> to __get_cpuid_

Ah.  Could you include stddef.h instead?  We avoid using stdio in tests
unless required for the specifics of the test, in order to be able to run
as most tests as possible against remote targets that don't support
remote I/O (which are most, though the local gdbserver testing boards nowadays
work around it).  Just #including <stdio.h> without relying on printf etc. itself
is probably not really problematic anywhere, but it's a good proxy
for identifying tests that wouldn't/shouldn't work in such targets.

Thanks,
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]