fix bugs where caller incorrectly released lock on error return path.

as a direct index into cpus.  Record apicid in struct cpu and have cpunum() look
for it. Replace cpu->id with cpunum() everywhere, and replace cpu->id with cpu->apicid.
Thanks to Xi Wang.

add to test that they indeed trap in user space
thanks to pablogventura@gmail.com and nwolovick@gmail.com

 for f in *.{h,c}; do sed -i .sed 's/[[:blank:]]*$//' $f; done
(Thanks to Nicolás Wolovick)