Re: [PATCH 2/2] ppc: kvm: use anon_inode_getfd() with O_CLOEXEC flag

2013-08-26 Thread Yann Droneaud
Le 26.08.2013 09:39, Paolo Bonzini a écrit : Il 25/08/2013 17:04, Alexander Graf ha scritto: On 24.08.2013, at 21:14, Yann Droneaud wrote: This patch set O_CLOEXEC flag on all file descriptors created with anon_inode_getfd() to not leak file descriptors across exec(). Signed-off-by: Yann

Re: [PATCH 03/13] infiniband: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-08 Thread Yann Droneaud
es already opened. [ In fact, supporting fork() is already a challenge for thoses API. ] So those file descriptors can safely be opened with O_CLOEXEC without disturbing users of InfiniBand verbs /RDMA ----8<-- Regards. -- Yann Droneaud OPTEYA -- To unsubs

Re: [PATCH 10/13] xfs: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-10 Thread Yann Droneaud
Hi, Le 09.07.2013 22:53, Ben Myers a écrit : On Mon, Jul 08, 2013 at 05:41:33PM -0500, Ben Myers wrote: On Tue, Jul 02, 2013 at 06:39:34PM +0200, Yann Droneaud wrote: > diff --git a/fs/xfs/xfs_ioctl.c b/fs/xfs/xfs_ioctl.c > index 5e99968..dc5b659 100644 > --- a/fs/xfs/xfs_ioctl.c &

Re: [PATCH v3 8/8] industrialio: use anon_inode_getfd() with O_CLOEXEC flag

2013-09-15 Thread Yann Droneaud
Hi, Le dimanche 15 septembre 2013 à 17:26 +0100, Jonathan Cameron a écrit : > On 09/06/13 11:39, Yann Droneaud wrote: [...] > > http://lkml.kernel.org/r/cover.1377372576.git.ydrone...@opteya.com > > > > Signed-off-by: Yann Droneaud > > Link: http://lkml.ke

[PATCH v3 0/8] Getting rid of get_unused_fd() / use O_CLOEXEC

2013-09-06 Thread Yann Droneaud
[SCTP] http://lkml.kernel.org/r/51d312e8.6090...@gmail.com http://lkml.kernel.org/r/20130702.161428.1703028286206350504.da...@davemloft.net [XFS] http://lkml.kernel.org/r/20130709205321.gv20...@sgi.com [PATCHSETv2] http://lkml.kernel.org/r/cover.1376327678.git.ydrone...@opteya.

[PATCH v3 1/8] ia64: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-09-06 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v3 7/8] file: remove get_unused_fd()

2013-09-06 Thread Yann Droneaud
be inherited across exec(). This patch removes get_unused_fd() so that newer kernel code use anon_inode_getfd() or get_unused_fd_flags() with flags provided by userspace. If flags cannot be given by userspace, O_CLOEXEC must be the default flag. Signed-off-by: Yann Droneaud Link: http

[PATCH v3 2/8] ppc/cell: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-09-06 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v3 6/8] events: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-09-06 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v3 5/8] fanotify: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-09-06 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v3 4/8] file: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-09-06 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v3 8/8] industrialio: use anon_inode_getfd() with O_CLOEXEC flag

2013-09-06 Thread Yann Droneaud
l.com/53603.html - [1] kvm: use anon_inode_getfd() with O_CLOEXEC flag http://lkml.kernel.org/r/cover.1377372576.git.ydrone...@opteya.com Signed-off-by: Yann Droneaud Link: http://lkml.kernel.org/r/cover.1378460926.git.ydrone...@opteya.com --- drivers/iio/industrialio-event.c | 2 +- 1 file ch

[PATCH v3 3/8] binfmt_misc: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-09-06 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH] MAINTAINERS: x...@oss.sgi.com is a list

2013-07-02 Thread Yann Droneaud
FILESYSTEM) linux-kernel@vger.kernel.org (open list) Signed-off-by: Yann Droneaud --- MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index ad7e322..7b6ab51 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -9139,7 +9139,6 @@ XFS FILESYSTEM P: Silicon

[PATCH] MAINTAINERS: dm-de...@redhat.com is a list

2013-07-02 Thread Yann Droneaud
...) +dm-de...@redhat.com (open list:DEVICE-MAPPER (LVM)) linux-r...@vger.kernel.org (open list:SOFTWARE RAID (Mu...) linux-kernel@vger.kernel.org (open list) Signed-off-by: Yann Droneaud --- MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 7b6ab51

[PATCH 00/13] Getting rid of get_unused_fd()

2013-07-02 Thread Yann Droneaud
et try to convert all calls to get_unused_fd() to get_unused_fd_flags(0) before removing get_unused_fd() macro. Without get_unused_fd() macro, more subsystems are likely to use anon_inode_getfd() and be teached to provide an API that let userspace choose the opening flags of the file descriptor. Y

[PATCH 13/13] file: remove get_unused_fd()

2013-07-02 Thread Yann Droneaud
be inherited across exec(). This patch removes get_unused_fd() so that newer kernel code use anon_inode_getfd() or get_unused_fd_flags() with flags provided by userspace. If flags cannot be given by userspace, O_CLOEXEC must be the default flag. Signed-off-by: Yann Droneaud --- include/linux

[PATCH 12/13] sctp: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- net/sctp/socket.c

[PATCH 11/13] events: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- kernel/events/core.c

[PATCH 06/13] vfio: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- drivers/vfio/vfio.c

[PATCH 08/13] file: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- fs/file.c | 2

[PATCH 03/13] infiniband: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- drivers/infin

[PATCH 02/13] ppc/cell: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- arch/powerpc/plat

[PATCH 10/13] xfs: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- fs/xfs/xfs_ioctl.c

[PATCH 04/13] android/sw_sync: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- drivers/stagi

[PATCH 09/13] fanotify: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- fs/notif

[PATCH 01/13] ia64: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- arch/ia64/kernel/per

[PATCH 05/13] android/sync: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- drivers/staging/andr

[PATCH 07/13] binfmt_misc: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-07-02 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud --- fs/binfmt_misc.c | 2

Re: [PATCH] MAINTAINERS: x...@oss.sgi.com is a list

2013-07-03 Thread Yann Droneaud
Hi, Le 03.07.2013 08:40, Dave Chinner a écrit : On Tue, Jul 02, 2013 at 05:00:47PM +0200, Yann Droneaud wrote: This patch changes type of x...@oss.sgi.com The output of ./scripts/get_maintainer.pl is modified to report x...@oss.sgi.com as a list: What's the problem with that? Al

Re: [PATCH] MAINTAINERS: x...@oss.sgi.com is a list

2013-07-03 Thread Yann Droneaud
Le 03.07.2013 11:24, Dave Chinner a écrit : On Wed, Jul 03, 2013 at 10:14:41AM +0200, Yann Droneaud wrote: Le 03.07.2013 08:40, Dave Chinner a écrit : >On Tue, Jul 02, 2013 at 05:00:47PM +0200, Yann Droneaud wrote: >>This patch changes type of x...@oss.sgi.com >> >>Th

Re: [PATCH] MAINTAINERS: x...@oss.sgi.com is a list

2013-07-03 Thread Yann Droneaud
Le 03.07.2013 11:50, Dave Chinner a écrit : On Wed, Jul 03, 2013 at 11:36:39AM +0200, Yann Droneaud wrote: Le 03.07.2013 11:24, Dave Chinner a écrit : >On Wed, Jul 03, 2013 at 10:14:41AM +0200, Yann Droneaud wrote: >>Le 03.07.2013 08:40, Dave Chinner a écrit : >>>On Tue, Jul

[PATCH v2 01/10] ia64: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-08-15 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v2 02/10] ppc/cell: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-08-15 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v2 05/10] vfio: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-08-15 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v2 04/10] android/sync: use get_unused_fd_flags(O_CLOEXEC) instead of get_unused_fd()

2013-08-15 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with call to get_unused_fd_flags(O_CLOEXEC) following advice from Erik Gilling. Signed-off-by: Yann Droneaud Cc: Erik Gilling Cc: Colin Cross Link: http://lkml.kernel.org/r/cacsp8sjxgmk2_kx_+rgzqqqwqkernvf1wt3k5tw991w5dfa...@mail.gmail.com Link: http://lkml.

[PATCH v2 07/10] file: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-08-15 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v2 08/10] fanotify: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-08-15 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v2 03/10] android/sw_sync: use get_unused_fd_flags(O_CLOEXEC) instead of get_unused_fd()

2013-08-15 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with call to get_unused_fd_flags(O_CLOEXEC) following advice from Erik Gilling. Signed-off-by: Yann Droneaud Cc: Erik Gilling Cc: Colin Cross Link: http://lkml.kernel.org/r/CACSP8SjZcpcpEtQHzcGYhf-MP7QGo0XpN7-uN7rmD=vNtopG=w...@mail.gmail.com

[PATCH v2 06/10] binfmt_misc: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-08-15 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH v2 10/10] file: remove get_unused_fd()

2013-08-15 Thread Yann Droneaud
be inherited across exec(). This patch removes get_unused_fd() so that newer kernel code use anon_inode_getfd() or get_unused_fd_flags() with flags provided by userspace. If flags cannot be given by userspace, O_CLOEXEC must be the default flag. Signed-off-by: Yann Droneaud Link: http

[PATCH v2 00/10] Getting rid of get_unused_fd_flags()

2013-08-15 Thread Yann Droneaud
ad of get_unused_fd() DROPPED: applied asis by subsystem maintainer. - sctp: use get_unused_fd_flags(0) instead of get_unused_fd() DROPPED: applied asis by subsystem maintainer. Yann Droneaud (10): ia64: use get_unused_fd_flags(0) instead of get_unused_fd() ppc/cell: use get_unused_fd_flags(0) i

[PATCH v2 09/10] events: use get_unused_fd_flags(0) instead of get_unused_fd()

2013-08-15 Thread Yann Droneaud
This patch replaces calls to get_unused_fd() with equivalent call to get_unused_fd_flags(0) to preserve current behavor for existing code. The hard coded flag value (0) should be reviewed on a per-subsystem basis, and, if possible, set to O_CLOEXEC. Signed-off-by: Yann Droneaud Link: http://lkml.kernel.o

[PATCH] USB: serial: fix stringify operator in usb-serial-simple

2013-08-18 Thread Yann Droneaud
From: Yann Droneaud usb-serial-simple uses an unknown stringify macro that make all drivers being named "stringify(vendor)". This can be a problem when two drivers have the same (wrong) name: kernel: usbcore: registered new interface driver usb_serial_simple kernel: usbs

Re: [PATCH] USB: serial: fix stringify operator in usb-serial-simple

2013-08-18 Thread Yann Droneaud
Le 18.08.2013 21:40, Greg Kroah-Hartman a écrit : On Sun, Aug 18, 2013 at 09:29:00PM +0200, Yann Droneaud wrote: Ugh, sorry about that, I thought there used to be a stringify() macro that used to do this. Nice patch, I'll queue it up. That's __stringify() which is defined in but:

[PATCH 1/2] kvm: use anon_inode_getfd() with O_CLOEXEC flag

2013-08-24 Thread Yann Droneaud
ed to choose, without race, if the file descriptor is going to be inherited across exec(). This patch set O_CLOEXEC flag on all file descriptors created with anon_inode_getfd() to not leak file descriptors across exec(). Signed-off-by: Yann Droneaud Link: http://lkml.kernel.org/r/cover.1377372576.git.y

[PATCH 0/2] kvm: use anon_inode_getfd() with O_CLOEXEC flag

2013-08-24 Thread Yann Droneaud
t.home [3] http://lkml.kernel.org/r/20130822171744.1297.13711.st...@bling.home Yann Droneaud (2): kvm: use anon_inode_getfd() with O_CLOEXEC flag ppc: kvm: use anon_inode_getfd() with O_CLOEXEC flag arch/powerpc/kvm/book3s_64_mmu_hv.c | 2 +- arch/powerpc/kvm/book3s_64_vio.c| 2 +- arch/powerpc/kvm/

[PATCH 2/2] ppc: kvm: use anon_inode_getfd() with O_CLOEXEC flag

2013-08-24 Thread Yann Droneaud
ed to choose, without race, if the file descriptor is going to be inherited across exec(). This patch set O_CLOEXEC flag on all file descriptors created with anon_inode_getfd() to not leak file descriptors across exec(). Signed-off-by: Yann Droneaud Link: http://lkml.kernel.org/r/cover.1377372576.git.y

strange bug, alloca suspected

2001-02-13 Thread Yann Droneaud
iners. >Submitter-Id: net >Originator:Yann Droneaud <[EMAIL PROTECTED]> >Organization: no others than mine >Confidential: no >Synopsis: undetermined >Severity: non-critical >Priority: low >Category: libc >Class: sw-bug >Releas

check_dis(c|k)_change(|d): duplicated code

2001-01-27 Thread Yann Droneaud
27;s a function 'check_disk_change' and in 'fs/devfs/base.c' there's a function 'check_disc_changed' They done exactly the same job. I think that one of these must be rewritted to call the other and it's not to me to tell you which one :-). -- Yann

fs block/devfs check disc change

2001-01-31 Thread Yann DRONEAUD
e of these must be rewritted to call/use the other and it's not to me to tell you which one :-). -- Yann Droneaud <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH v2 1/3] coccinelle: also catch kzfree() issues

2016-06-21 Thread Yann Droneaud
Hi, Le mardi 21 juin 2016 à 11:43 +0200, Michal Marek a écrit : > Dne 20.6.2016 v 22:21 Julia Lawall napsal(a): > > On Mon, 20 Jun 2016, Michal Marek wrote: > > On 2016-05-23 17:18, Julia Lawall wrote: > > > > On Mon, 23 May 2016, Yann Droneaud wrote: > > > &g

Re: [kernel-hardening] [RFC patch 1/6] random: Simplify API for random address requests

2016-07-27 Thread Yann Droneaud
Hi, Le mardi 26 juillet 2016 à 03:01 +, Jason Cooper a écrit : > To date, all callers of randomize_range() have set the length to 0, > and check for a zero return value.  For the current callers, the only > way to get zero returned is if end <= start.  Since they are all > adding a constant to

Re: [PATCH 1/7] random: Simplify API for random address requests

2016-07-29 Thread Yann Droneaud
-35,6 +35,7 @@ extern const struct file_operations random_fops, > urandom_fops; >  unsigned int get_random_int(void); >  unsigned long get_random_long(void); >  unsigned long randomize_range(unsigned long start, unsigned long end, > unsigned long len); > +unsigned long randomize_addr(unsigned long start, unsigned long range); >   >  u32 prandom_u32(void); >  void prandom_bytes(void *buf, size_t nbytes); Regards. -- Yann Droneaud OPTEYA

Re: [PATCH] timerfd: expose uapi header

2015-10-05 Thread Yann Droneaud
ollide with O_* ones. We want > + * to re-use O_* flags that couldn't possibly have a meaning > + * from eventfd, in order to leave a free define-space for > + * shared O_* flags. > + */ > +#define TFD_TIMER_ABSTIME (1 << 0) > +#define TFD_TIMER_CANCEL_ON_SET

Re: [PATCH v2 3/3] falloc: expose ioctl numbers into uapi

2015-10-06 Thread Yann Droneaud
ved area */ > +}; > + > +#define FS_IOC_RESVSP_IOW('X', 40, struct > space_resv) > +#define FS_IOC_RESVSP64 _IOW('X', 42, struct > space_resv) > + > #endif /* _UAPI_FALLOC_H_ */ Regards. -- Yann Droneaud OPTEYA -- To unsubscribe

Re: [PATCH] IB/core: avoid 32-bit warning

2015-10-07 Thread Yann Droneaud
d-off-by: Arnd Bergmann > Fixes: 565197dd8fb1 ("IB/core: Extend ib_uverbs_create_cq") > Reviewed-by: Yann Droneaud > diff --git a/drivers/infiniband/core/uverbs_cmd.c > b/drivers/infiniband/core/uverbs_cmd.c > index be4cb9f04be3..88b3b78340f2 100644 > --- a/drivers

Re: [PATCH] IB/core: avoid 32-bit warning

2015-10-07 Thread Yann Droneaud
esp)); > > Would it make sense to cast inside INIT_UDATA() and not have callers > worry about it? It's ... complicated. See INIT_UDATA_BUF_OR_NULL(). Awayway, I have patch to do the opposite, eg. explicitly cast u64 value to (void __user *)(unsigned long) in the caller functi

[-stable] perf/x86: Fix copy_from_user_nmi() return if range is not ok

2015-09-08 Thread Yann Droneaud
Hi, I believe commit ebf2d2689de551d90965090bb991fc640a0c0d41 ("perf/x86: Fix copy_from_user_nmi() return if range is not ok") should be applied to stable kernels starting from v3.13.y up to v4.1.y. Regards. -- Yann Droneaud OPTEYA -- To unsubscribe from this list: send the line &q

[PATCH v2 0/2] perf tool: improve error handling in perf_flag_probe()

2015-10-01 Thread Yann Droneaud
t. [1] http://lkml.kernel.org/r/1406908014-8312-1-git-send-email-jo...@kernel.org [2] http://lkml.kernel.org/r/cover.1425901229.git.ydrone...@opteya.com [3] http://lkml.kernel.org/r/cover.1410595700.git.ydrone...@opteya.com [4] http://lkml.kernel.org/r/20140920121438.gb15...@krava.brq.redhat.com Y

[PATCH v2 1/2] perf tools: shortcut PERF_FLAG_FD_CLOEXEC probing in case of EBUSY error

2015-10-01 Thread Yann Droneaud
error. Cc: Adrian Hunter Cc: David Ahern Cc: Frederic Weisbecker Cc: Jiri Olsa Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc: Stephane Eranian Cc: William Cohen Link: http://lkml.kernel.org/r/cover.1443708092.git.ydrone...@opteya.com Signed-off-by: Yann Droneaud --- tool

[PATCH v2 2/2] perf tools: report PERF_FLAG_FD_CLOEXEC probing error once

2015-10-01 Thread Yann Droneaud
Jiri Olsa Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Cc: Stephane Eranian Cc: William Cohen Link: http://lkml.kernel.org/r/cover.1443708092.git.ydrone...@opteya.com Reported-by: Jiri Olsa Signed-off-by: Yann Droneaud --- tools/perf/util/cloexec.c | 25 + 1

Re: [PATCH] uapi: remove i2o header in uapi

2015-10-01 Thread Yann Droneaud
y += i2c.h -header-y += i2o-dev.h header-y += i8k.h header-y += icmp.h header-y += icmpv6.h Note, you may also use git format-patch -D to reduce the size of your patch. Regards. -- Yann Droneaud OPTEYA -- To unsubscribe from this list: send the line "unsubscribe linux-kern

Re: [PATCH v2 1/2] perf tools: shortcut PERF_FLAG_FD_CLOEXEC probing in case of EBUSY error

2015-10-01 Thread Yann Droneaud
Hi, Le jeudi 01 octobre 2015 à 22:52 +0200, Jiri Olsa a écrit : > On Thu, Oct 01, 2015 at 05:16:25PM +0200, Yann Droneaud wrote: > > This patch is a simplification of the logic introduced as part of > > commit 63914aca8f7e ('perf tools: Show better error message in case

Re: [PATCH v2 2/2] perf tools: report PERF_FLAG_FD_CLOEXEC probing error once

2015-10-02 Thread Yann Droneaud
Le vendredi 02 octobre 2015 à 15:20 +0200, Jiri Olsa a écrit : > On Thu, Oct 01, 2015 at 05:16:26PM +0200, Yann Droneaud wrote: > > In case of failure, unrelated to PERF_FLAG_FD_CLOEXEC, > > perf_flag_probe() reports the error twice. For example: > > > > $

Re: [RFC PATCH] UAPI: Document auxvec AT_* namespace policy and note reservations

2018-05-17 Thread Yann Droneaud
21be327e1ea4cd1a3e7;hb=23158b08a0908f381459f273a984c6fd3283 63cb#l142 And there's also AT_ANYNET and AT_ANYNODE from ddp (aka. AppleTalk) http://man7.org/linux/man-pages/man7/ddp.7.html Regards. -- Yann Droneaud OPTEYA

Re: [PATCH] kbuild: remove old information in headers_install.txt document

2014-12-05 Thread Yann Droneaud
following a discussion with Sam Ravnborg[2]. See the mail thread[3]. Regards. [1] http://lkml.kernel.org/r/b5181cf7604baa454c11f7aa92d07dd05349ce46.1410712841.git.ydrone...@opteya.com [2] http://lkml.kernel.org/r/20140713212608.gb16...@ravnborg.org [3] http://lkml.kernel.org/g/cover.1410712841.git.ydrone...@opteya.com

Re: CVE-2014-8159 kernel: infiniband: uverbs: unprotected physical memory access

2015-04-02 Thread Yann Droneaud
NVAL); > + Can access_ok() be used here ? if (!access_ok(writable ? VERIFY_WRITE : VERIFY_READ, addr, size)) return ERR_PTR(-EINVAL); > if (!can_do_mlock()) > return ERR_PTR(-EPERM); > Regards. -- Yann Droneaud OPTEYA

Re: CVE-2014-8159 kernel: infiniband: uverbs: unprotected physical memory access

2015-04-02 Thread Yann Droneaud
Hi, Le jeudi 02 avril 2015 à 10:52 +, Shachar Raindel a écrit : > > -Original Message- > > From: Yann Droneaud [mailto:ydrone...@opteya.com] > > Sent: Thursday, April 02, 2015 1:05 PM > > Le mercredi 18 mars 2015 à 17:39 +, Shachar Raindel a écrit : >

Re: CVE-2014-8159 kernel: infiniband: uverbs: unprotected physical memory access

2015-04-02 Thread Yann Droneaud
Hi, Le jeudi 02 avril 2015 à 10:52 +, Shachar Raindel a écrit : > > -Original Message- > > From: Yann Droneaud [mailto:ydrone...@opteya.com] > > Sent: Thursday, April 02, 2015 1:05 PM > > Le mercredi 18 mars 2015 à 17:39 +, Shachar Raindel a écrit : ... >

Re: CVE-2014-8159 kernel: infiniband: uverbs: unprotected physical memory access

2015-04-02 Thread Yann Droneaud
Hi Haggai, Le jeudi 02 avril 2015 à 18:18 +0300, Haggai Eran a écrit : > On 02/04/2015 16:30, Yann Droneaud wrote: > > Hi, > > > > Le jeudi 02 avril 2015 à 10:52 +, Shachar Raindel a écrit : > >>> -Original Message- > >>> From: Yann Dron

Re: CVE-2014-8159 kernel: infiniband: uverbs: unprotected physical memory access

2015-04-02 Thread Yann Droneaud
Hi, Le jeudi 02 avril 2015 à 16:44 +, Shachar Raindel a écrit : > > -Original Message- > > From: Yann Droneaud [mailto:ydrone...@opteya.com] > > Sent: Thursday, April 02, 2015 7:35 PM > > Another related question: as the large memory range could be registere

Re: CVE-2014-8159 kernel: infiniband: uverbs: unprotected physical memory access

2015-04-03 Thread Yann Droneaud
Hi, Le vendredi 03 avril 2015 à 08:39 +, Haggai Eran a écrit : > On Thursday, April 2, 2015 11:40 PM, Yann Droneaud > wrote: > > Le jeudi 02 avril 2015 à 16:44 +, Shachar Raindel a écrit : > >> > -Original Message- > >> > From: Yann D

Re: [PATCH linux-next 1/4] infiniband/ipoib: fix possible NULL pointer dereference in ipoib_get_iflink

2015-04-14 Thread Yann Droneaud
ition or not, I build linux-next-20150414 on two machines. Both > > machines panic at modprobe ib_ipoib. Do you means I need to report a > > bug? But I do not know report to who or where. > > Here is the tag: > Fixes: 5aa7add8f14b ("infiniband/ipoib: implement ndo_get_iflink&quo

Re: CVE-2014-8159 kernel: infiniband: uverbs: unprotected physical memory access

2015-04-08 Thread Yann Droneaud
Hi, Le jeudi 02 avril 2015 à 16:34 +, Shachar Raindel a écrit : > > -Original Message- > > From: Yann Droneaud [mailto:ydrone...@opteya.com] > > Sent: Thursday, April 02, 2015 6:16 PM > > Le jeudi 02 avril 2015 à 10:52 +, Shachar Raindel a écrit : >

Re: CVE-2014-8159 kernel: infiniband: uverbs: unprotected physical memory access

2015-04-08 Thread Yann Droneaud
Hi, Le mercredi 08 avril 2015 à 14:19 +0200, Yann Droneaud a écrit : > Le jeudi 02 avril 2015 à 16:34 +, Shachar Raindel a écrit : > > > -Original Message- > > > From: Yann Droneaud [mailto:ydrone...@opteya.com] > > > Sent: Thursday, April 02, 2015 6:16 P

Re: CVE-2014-8159 kernel: infiniband: uverbs: unprotected physical memory access

2015-04-13 Thread Yann Droneaud
Hi, Le jeudi 02 avril 2015 à 18:12 +, Haggai Eran a écrit : > On Thursday, April 2, 2015 7:44 PM, Shachar Raindel wrote: > >> -Original Message- > >> From: Yann Droneaud [mailto:ydrone...@opteya.com] > >> Le jeudi 02 avril 2015 à 18:18 +0300, Haggai Eran

[PATCH] x86: fix copy_from_user_nmi() return if range is not ok

2015-06-22 Thread Yann Droneaud
copied, eg. 0, instead of the remaining bytes. As all users of copy_from_user_nmi() were modified as part of commit 0a196848ca36, the function should be fixed to return the total number of bytes if range is not correct. Signed-off-by: Yann Droneaud --- arch/x86/lib/usercopy.c | 2 +- 1 file

[PATCH v1 0/3] No more seq_file pre-allocation

2015-06-01 Thread Yann Droneaud
seq_open(). Changes from v0 [0]: - convert kmalloc() + memset() to kzalloc() - revised a bit commit messages [0] [PATCH 0/3] seq_file allocation in seq_open() http://lkml.kernel.org/r/cover.1430777196.git.ydrone...@opteya.com http://lkml.kernel.org/g/cover.1430777196.git.ydrone...@opteya.com Yann Dro

[PATCH v1 2/3] fs: allocate structure unconditionally in seq_open()

2015-06-01 Thread Yann Droneaud
q_open(). Link: http://lkml.kernel.org/r/cover.1433193673.git.ydrone...@opteya.com Signed-off-by: Yann Droneaud --- fs/seq_file.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/fs/seq_file.c b/fs/seq_file.c index c14f6a43beb5..a909f12dad4d 100644 --- a/fs/seq_file.

[PATCH v1 3/3] fs: documents seq_open()'s usage of file->private_data

2015-06-01 Thread Yann Droneaud
seq_open() store its struct seq_file in file->private_data, thus, it must not be modified by user of seq_file. Link: http://lkml.kernel.org/r/cover.1433193673.git.ydrone...@opteya.com Signed-off-by: Yann Droneaud --- fs/seq_file.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a

[PATCH v1 1/3] fs: use seq_open_private() for proc_mounts

2015-06-01 Thread Yann Droneaud
igned-off-by: Yann Droneaud --- fs/mount.h | 3 --- fs/namespace.c | 6 +++--- fs/proc_namespace.c | 34 -- 3 files changed, 19 insertions(+), 24 deletions(-) diff --git a/fs/mount.h b/fs/mount.h index b5b8082bfa42..14db05d424f7 100644 --- a/fs/moun

Re: [PATCH 2/4] pid: add pidfd_open()

2019-03-27 Thread Yann Droneaud
FD_TO_PIDFD) > + return -EINVAL; > + > + if (pid != -1) > + return -EINVAL; > + > + if (pidfd >= 0) > I think it can be stricter with: if (pidfd != -1) (and match the check done for flag == 0). Regards. -- Yann Droneaud OPTEYA

[PATCH 4/4] random/trace: remove unused trace_xfer_secondary_pool()

2020-05-21 Thread Yann Droneaud
Since commit 84df7cdfbb21 ('random: delete code to pull data into pools'), trace_xfer_secondary_pool() is unused. Cc: Andy Lutomirski Cc: Theodore Ts'o Signed-off-by: Yann Droneaud --- include/trace/events/random.h | 29 - 1 file changed, 29 del

[PATCH 2/4] random/trace: remove unused trace_push_to_pool()

2020-05-21 Thread Yann Droneaud
Since commit 90ea1c6436d2 ('random: remove the blocking pool') trace_push_to_pool() is unused. Cc: Andy Lutomirski Cc: Theodore Ts'o Signed-off-by: Yann Droneaud --- include/trace/events/random.h | 22 -- 1 file changed, 22 deletions(-) diff --git a/inclu

[PATCH 0/4] random/trace: remove unused function

2020-05-21 Thread Yann Droneaud
Hi, After simplifications brought recently to random [1], some trace functions are no more needed. This patchset removes them. [1] https://lore.kernel.org/linux-api/cover.1577088521.git.l...@kernel.org/ Yann Droneaud (4): random/trace: remove unused trace_random_read() random/trace: remove

[PATCH 3/4] random/trace: remove unused trace_extract_entropy_user()

2020-05-21 Thread Yann Droneaud
Since commit 90ea1c6436d2 ('random: remove the blocking pool') trace_extract_entropy_user() is unused. As the result, trace_extract_entropy() is the only trace function needed, thus there's no need for a separate event class / event. Cc: Andy Lutomirski Cc: Theodore Ts'o

[PATCH 1/4] random/trace: remove unused trace_random_read()

2020-05-21 Thread Yann Droneaud
Since commit 30c08efec888 ('random: make /dev/random be almost like /dev/urandom'), trace_random_read() is unused. Cc: Andy Lutomirski Cc: Theodore Ts'o Signed-off-by: Yann Droneaud --- include/trace/events/random.h | 25 - 1 file changed, 25 deletions

Re: [PATCH] events: add a flag to perf_event_open() to set O_CLOEXEC

2013-11-04 Thread Yann Droneaud
Hi, Le jeudi 31 octobre 2013 à 19:12 +0100, Peter Zijlstra a écrit : > On Wed, Oct 30, 2013 at 10:35:50PM +0100, Yann Droneaud wrote: > > This patch adds PERF_FLAG_FD_CLOEXEC flag for > > perf_event_open() syscall. > > > > perf_event_open() creates a new file desc

[PATCH] 6lowpan: add a license to 6lowpan_iphc module

2014-01-22 Thread Yann Droneaud
the basic MODULE_LICENSE(); with GPL license: the code was copied from net/ieee802154/6lowpan.c which is GPL and the module exports symbol with EXPORT_SYMBOL_GPL();. Cc: Jukka Rissanen Cc: Alexander Aring Cc: Marcel Holtmann Signed-off-by: Yann Droneaud --- net/ieee802154/6lowpan_iphc.c | 3 +

[PATCHv3] driver core/platform: don't leak memory allocated for dma_mask

2014-01-26 Thread Yann Droneaud
/patchwork.kernel.org/patch/3480961/ [3] http://lkml.kernel.org/r/1386886207-2735-1-git-send-email-ydrone...@opteya.com Cc: Uwe Kleine-König Cc: Dmitry Torokhov Cc: Greg Kroah-Hartman Signed-off-by: Yann Droneaud --- Hi Greg and Uwe, I've tried to add dma_mask to platform_object, and fou

[PATCH 0/2] Revamp randomize_range()

2014-01-26 Thread Yann Droneaud
27;ve not added the maintainers of the various arch modified by the patches, wanted first to have your opinion about the changes. Regards. Yann Droneaud (2): random: remove unused len argument in randomize_range() function random: don't return 0 in randomize_range() arch/arm/kernel

[PATCH 2/2] random: don't return 0 in randomize_range()

2014-01-26 Thread Yann Droneaud
.kernel.org/r/cover.1390770607.git.ydrone...@opteya.com Cc: Theodore Ts'o Signed-off-by: Yann Droneaud --- arch/arm/kernel/process.c| 2 +- arch/tile/mm/mmap.c | 2 +- arch/x86/kernel/process.c| 2 +- arch/x86/kernel/sys_x86_64.c | 5 + drivers/char/random.c| 4 +++-

[PATCHv3] perf tools: enable close-on-exec flag on perf file descriptor

2014-01-26 Thread Yann Droneaud
1/ Cc: Peter Zijlstra Cc: Arnaldo Carvalho de Melo Cc: Jiri Olsa Cc: Andi Kleen Cc: Ingo Molnar Signed-off-by: Yann Droneaud --- tools/perf/Makefile.perf | 1 + tools/perf/bench/mem-memcpy.c | 4 ++- tools/perf/bench/mem-memset.c | 4 ++- tools/perf/builtin-sche

[PATCH 1/2] random: remove unused len argument in randomize_range() function

2014-01-26 Thread Yann Droneaud
70607.git.ydrone...@opteya.com Cc: Theodore Ts'o Signed-off-by: Yann Droneaud --- arch/arm/kernel/process.c | 2 +- arch/arm64/kernel/process.c | 2 +- arch/tile/mm/mmap.c | 2 +- arch/unicore32/kernel/process.c | 2 +- arch/x86/kernel/process.c | 2 +- arch/

[PATCHv4] driver core/platform: don't leak memory allocated for dma_mask

2014-01-27 Thread Yann Droneaud
g/r/1389649085-7365-1-git-send-email-ydrone...@opteya.com https://patchwork.kernel.org/patch/3480961/ [4] http://lkml.kernel.org/r/1386886207-2735-1-git-send-email-ydrone...@opteya.com Cc: Uwe Kleine-König Cc: Dmitry Torokhov Cc: Greg Kroah-Hartman Signed-off-by: Yann Droneaud --- Hi

cscope: issue with symlinks in tools/testing/selftests/powerpc/copyloops/

2014-04-03 Thread Yann Droneaud
version 15.7a does not follow symbolic links http://sourceforge.net/p/cscope/bugs/261/ Regards. -- Yann Droneaud OPTEYA -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger

[PATCH 1/2] perf tools: remove unused test-volatile-register-var.c

2014-01-11 Thread Yann Droneaud
Since commit 01287e2cb7ad, test-volatile-register-var.c is no more built as part of the automatic feature check. This patch remove the unneeded file. Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Cc: Peter Zijlstra Cc: David Ahern Cc: Jiri Olsa Cc: Namhyung Kim Signed-off-by: Yann Droneaud

[PATCH 2/2] perf tools: add .gitignore for files generated for feature check

2014-01-11 Thread Yann Droneaud
As part of system features auto-detection, some dependency files are generated and some programs are built. These files should not be considered by git, so this patch adds a dedicated .gitignore in tools/perf/config/ sub-directory to ignore the files. Signed-off-by: Yann Droneaud --- tools

[PATCH 0/2] perf: little build clean up

2014-01-11 Thread Yann Droneaud
Hi, Please find 2 little patches to remove an unused file and to ignore files generated as part of the feature auto-detection. Regards. Yann Droneaud (2): perf tools: remove unused test-volatile-register-var.c perf tools: add .gitignore for files generated for feature check tools/perf

  1   2   3   >