Rename aiocblist to kaiocb and use consistent variable names.

[jhb]

Summary

Rename aiocblist to kaiocb and use consistent variable names.

Typically <foo>list is used for a structure that holds a list head in
FreeBSD, not for members of a list. As such, rename 'struct aiocblist'
to 'struct kaiocb' (the kernel version of 'struct aiocb').

While here, use more consistent variable names for AIO control blocks:

• Use 'job' instead of 'aiocbe', 'cb', 'cbe', or 'iocb' for kernel job objects.
• Use 'jobn' instead of 'cbn' for use with TAILQ_FOREACH_SAFE().
• Use 'sjob' and 'sjobn' instead of 'scb' and 'scbn' for fsync jobs.
• Use 'ujob' instead of 'aiocbp', 'job', 'uaiocb', or 'uuaiocb' to hold a user pointer to a 'struct aiocb'.
• Use 'ujobp' instead of 'aiocbp' for a user pointer to a 'struct aiocb *'.

This is perhaps excessive (the various variable names in particular),
but it is more consistent IMO.

Test Plan

• Build tested.

[kib]

sys/kern/vfs_aio.c

250 ↗

(On Diff #12851)

There shoul be tabs after define.

496 ↗

(On Diff #12851)

Might be, lowercase zone names ?

949 ↗

(On Diff #12851)

Since you are changing style, perhaps remove initialization from declarations ?

[jhb]

sys/kern/vfs_aio.c

496 ↗

(On Diff #12851)

For now I will leave these as-is to avoid noise in the diff. Looking in vmstat -z we do generally seem to prefer lower case zone names, but we do have some upper case names. I think the upper case names are actually places that used to use the old zone allocator pre-UMA.

[jhb]

• Fix style on lines that are changing anyway.