cvs commit: src/sys/sys aio.h signal.h signalvar.h src/sys/kern syscalls.master vfs_aio.c

David Xu davidxu at
Sat Nov 22 01:18:25 PST 2008

John Baldwin wrote:
> On Saturday 29 October 2005 10:12:49 pm David Xu wrote:
>> davidxu     2005-10-30 02:12:49 UTC
>>   FreeBSD src repository
>>   Modified files:
>>     sys/sys              signal.h signalvar.h aio.h 
>>     sys/kern             syscalls.master vfs_aio.c 
>>   Log:
>>   Fix sigevent's POSIX incompatible problem by adding member fields
>>   sigev_notify_function and sigev_notify_attributes. AIO syscalls
>>   use sigevent, so they have to be adjusted.
>>   Reviewed by:    alc
> I'm looking at doing 32-bit compat for AIO, and the __spare__ thing in 'struct 
> aiocb' strikes me as rather odd.  For one, why not use 'struct osigevent 
> __spare__;' instead of the explicit sizeof()'s?  The sizeof's make 
> assumptions about padding for example. 
If you have patch, please commit it.

>  Secondly, since you bumped the 
> syscalls for the new structure, you didn't need the __spare__ hack at all.
I had put spare fields, I didn't know if people will need spare fields 
in future
without breaking binary compatibility, so I just reused the old fields.

David Xu

More information about the cvs-src mailing list