This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc 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] |
Hi Joseph, On 03/12/2014 08:14 AM, David Holsgrove wrote: > Hi Joseph, > >> -----Original Message----- >> From: Joseph Myers [mailto:joseph@codesourcery.com] >> Sent: Sunday, 2 March 2014 8:56 am >> To: libc-alpha@sourceware.org; David Holsgrove >> Subject: Re: Fix __ASSUME_PSELECT for MicroBlaze (bug 16642) >> >> On Thu, 27 Feb 2014, Joseph S. Myers wrote: >> >>> This patch makes the MicroBlaze kernel-features.h undefine >>> __ASSUME_PSELECT. I'd also encourage wiring it up in the kernel (so >>> you can then make this #undef conditional, and eventually obsolete >>> once a recent-enough kernel is required). I suspect it wasn't wired >>> up because of the mistaken comment in asm/unistd.h "obsolete -> >>> sys_pselect7" (there is no such syscall as pselect7). >> >> It's come to my attention in this review that preadv and pwritev also >> still aren't wired up in the MicroBlaze syscall table. I'll defer a glibc >> patch for the corresponding __ASSUME_* (bug 16649) until the present patch >> is reviewed. David, I'd also encourage wiring up preadv and pwritev in >> the kernel - and reviewing the MicroBlaze syscall table for any more cases >> where syscalls aren't wired up (whether or not the __NR_* values are >> assigned), in case there are some that don't have corresponding glibc >> __ASSUME_* macros and so won't be caught by review of the definitions in >> glibc. >> > > Thanks very much for the patch, and for pointing out that miss. > > The Microblaze Kernel maintainer (Michal Simek) and I will have a chat and review > the syscall table for Microblaze. I believe Michal has a kernel patch pending to > address pselect already. I have wired them in xilinx git tree and also added missing sched_setattr and sched_getattr and all these patches are queued for mainline kernel v3.15 and they will be available in linux-next tomorrow. I am not sure how glibc handle this but pselect6, preadv and pwritev were listed in unistd.h that's why there shouldn't be any problem with glibc setup. It means it is up to you if you want to add any temporary solution to repository when this will be available soon. Definitely thanks for syscall reviewing. Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/ Maintainer of Linux kernel - Xilinx Zynq ARM architecture Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
Attachment:
signature.asc
Description: OpenPGP digital signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |