[COMMITTED PATCH] Consolidate not-cancel.h files.
Roland McGrath
roland@hack.frob.com
Wed May 14 21:29:00 GMT 2014
> On Wed, 14 May 2014, Roland McGrath wrote:
>
> > There are in fact only two versions of not-cancel.h actually in use: the
> > sysdeps/generic one for non-Linux, and the NPTL one living in
> > nptl/sysdeps/unix/sysv/linux/i386/. The diff looks nontrivial only because
> > moving nptl/sysdeps/unix/sysv/linux/i386/not-cancel.h verbatim to
> > sysdeps/unix/sysv/linux/ clobbered the unused file that was there before.
>
> I don't believe that file was unused; plenty of architectures didn't have
> their own not-cancel.h files and so used the sysdeps/unix/sysv/linux/ one.
> It may well be that the differences were *unnecessary*; I haven't thought
> what what this file should optimally look like.
You're right. I misread the set of files. Sorry about that.
The machines that were using the old sysdeps/unix/sysv/linux/not-cancel.h
are:
alpha
arm
hppa
ia64
microblaze
mips
mips64
The difference between the old and new files is that the old one used
INLINE_SYSCALL for everything, while the new one uses the __*_nocancel
entry points that are defined along with the ABI entry points by the
syscall macros.
I think we want uniformity across machines between those two choices,
though conceivably there is a different optimization choice to be made on
some machines than others. My inclination is to leave things as they are
now and just have everybody test that nothing broke. Does that sound OK?
Thanks,
Roland
More information about the Libc-alpha
mailing list