Re: [PATCH] [RFC] Remove bdflush syscall stub
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: Florian Weimer <fweimer@xxxxxxxxxx>
- Subject: Re: [PATCH] [RFC] Remove bdflush syscall stub
- From: Cyril Hrubis <chrubis@xxxxxxx>
- Date: Tue, 28 May 2019 13:20:23 +0200
- Cc: Andreas Schwab <schwab@xxxxxxxxxxxxxx>, lkml <linux-kernel@xxxxxxxxxxxxxxx>, linux-alpha@xxxxxxxxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, linux-ia64@xxxxxxxxxxxxxxx, linux-m68k@xxxxxxxxxxxxxxx, Michal Simek <monstr@xxxxxxxxx>, linux-mips@xxxxxxxxxxxxxxx, linux-parisc@xxxxxxxxxxxxxxx, linuxppc-dev@xxxxxxxxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx, linux-sh@xxxxxxxxxxxxxxx, sparclinux@xxxxxxxxxxxxxxx, linux-xtensa@xxxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, linux-api@xxxxxxxxxxxxxxx
- In-reply-to: <87ftoyg7t3.fsf@oldenburg2.str.redhat.com>
- References: <20190528101012.11402-1-chrubis@suse.cz> <mvmr28idgfu.fsf@linux-m68k.org> <20190528104017.GA11969@rei> <87ftoyg7t3.fsf@oldenburg2.str.redhat.com>
- User-agent: Mutt/1.10.1 (2018-07-13)
Hi!
> >> > I've tested the patch on i386. Before the patch calling bdflush() with
> >> > attempt to tune a variable returned 0 and after the patch the syscall
> >> > fails with EINVAL.
> >>
> >> Should be ENOSYS, doesn't it?
> >
> > My bad, the LTP syscall wrapper handles ENOSYS and produces skipped
> > results based on that.
> >
> > EINVAL is what you get for not yet implemented syscalls, i.e. new
> > syscall on old kernel.
>
> EINVAL? Is that a bdflush-specific thing, test-specific, or is itmore
> general?
>
> glibc has fallback paths that test for ENOSYS only. EINVAL will be
> passed to the application, skipping fallback. For missing system calls,
> this is not what we want.
The syscall returns ENOSYS after this change, sorry for the confusion.
--
Cyril Hrubis
chrubis@xxxxxxx
[Index of Archives]
[Linux Kernel]
[Sparc Linux]
[DCCP]
[Linux ARM]
[Yosemite News]
[Linux SCSI]
[Linux x86_64]
[Linux for Ham Radio]