From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Pieter Smith <pieter@boesman.nl>
Cc: Amit Shah <amit.shah@redhat.com>,
open list <linux-kernel@vger.kernel.org>,
Josh Triplett <josh@joshtriplett.org>,
Arnd Bergmann <arnd@arndb.de>,
"open list:VIRTIO CONSOLE DR..."
<virtualization@lists.linux-foundation.org>
Subject: Re: [PATCH 52/56] drivers/char/virtio: support compiling out splice
Date: Thu, 13 Nov 2014 14:09:49 -0800 [thread overview]
Message-ID: <20141113220949.GC546@kroah.com> (raw)
In-Reply-To: <1415913813-362-53-git-send-email-pieter@boesman.nl>
On Thu, Nov 13, 2014 at 10:23:29PM +0100, Pieter Smith wrote:
> Compile out splice support from virtio character driver when the splice-family
> of syscalls is not supported by the system (i.e. CONFIG_SYSCALL_SPLICE is
> undefined).
>
> Signed-off-by: Pieter Smith <pieter@boesman.nl>
> ---
> drivers/char/virtio_console.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c
> index b585b47..de5e2cb 100644
> --- a/drivers/char/virtio_console.c
> +++ b/drivers/char/virtio_console.c
> @@ -870,6 +870,7 @@ struct sg_list {
> struct scatterlist *sg;
> };
>
> +#ifdef CONFIG_SYSCALL_SPLICE
> static int pipe_to_sg(struct pipe_inode_info *pipe, struct pipe_buffer *buf,
> struct splice_desc *sd)
> {
> @@ -976,6 +977,7 @@ error_out:
> pipe_unlock(pipe);
> return ret;
> }
> +#endif /* #ifdef CONFIG_SYSCALL_SPLICE */
Not worth the #ifdef mess.
WARNING: multiple messages have this Message-ID (diff)
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Pieter Smith <pieter@boesman.nl>
Cc: Josh Triplett <josh@joshtriplett.org>,
Arnd Bergmann <arnd@arndb.de>, Amit Shah <amit.shah@redhat.com>,
"open list:VIRTIO CONSOLE DR..."
<virtualization@lists.linux-foundation.org>,
open list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 52/56] drivers/char/virtio: support compiling out splice
Date: Thu, 13 Nov 2014 14:09:49 -0800 [thread overview]
Message-ID: <20141113220949.GC546@kroah.com> (raw)
In-Reply-To: <1415913813-362-53-git-send-email-pieter@boesman.nl>
On Thu, Nov 13, 2014 at 10:23:29PM +0100, Pieter Smith wrote:
> Compile out splice support from virtio character driver when the splice-family
> of syscalls is not supported by the system (i.e. CONFIG_SYSCALL_SPLICE is
> undefined).
>
> Signed-off-by: Pieter Smith <pieter@boesman.nl>
> ---
> drivers/char/virtio_console.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c
> index b585b47..de5e2cb 100644
> --- a/drivers/char/virtio_console.c
> +++ b/drivers/char/virtio_console.c
> @@ -870,6 +870,7 @@ struct sg_list {
> struct scatterlist *sg;
> };
>
> +#ifdef CONFIG_SYSCALL_SPLICE
> static int pipe_to_sg(struct pipe_inode_info *pipe, struct pipe_buffer *buf,
> struct splice_desc *sd)
> {
> @@ -976,6 +977,7 @@ error_out:
> pipe_unlock(pipe);
> return ret;
> }
> +#endif /* #ifdef CONFIG_SYSCALL_SPLICE */
Not worth the #ifdef mess.
next prev parent reply other threads:[~2014-11-13 22:09 UTC|newest]
Thread overview: 87+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <pieter@boesman.nl>
[not found] ` <1415913813-362-1-git-send-email-pieter@boesman.nl>
2014-11-13 21:22 ` [PATCH 01/56] moved sendfile syscall to splice translation unit Pieter Smith
2014-11-13 21:22 ` [PATCH 02/56] moved kernel_write out of " Pieter Smith
[not found] ` <1415913813-362-1-git-send-email-pieter-qeJ+1H9vRZbz+pZb47iToQ@public.gmane.org>
2014-11-13 21:22 ` [PATCH 03/56] fs: Support compiling out splice-family syscalls Pieter Smith
2014-11-13 21:22 ` Pieter Smith
2014-11-13 21:22 ` Pieter Smith
2014-11-13 21:22 ` [PATCH 04/56] fs: Macros to define splice file_operations Pieter Smith
2014-11-13 21:49 ` Richard Weinberger
2014-11-13 22:24 ` josh
2014-11-13 21:51 ` Al Viro
2014-11-13 21:22 ` [PATCH 05/56] fs/lustre: support compiling out splice Pieter Smith
2014-11-13 22:09 ` Greg Kroah-Hartman
2014-11-13 21:22 ` [PATCH 06/56] fs/adfs: " Pieter Smith
2014-11-13 21:22 ` [PATCH 07/56] fs/affs: " Pieter Smith
2014-11-13 21:22 ` [PATCH 08/56] fs/afs: " Pieter Smith
2014-11-13 21:22 ` [PATCH 09/56] fs/bad_inode: " Pieter Smith
2014-11-13 21:22 ` [PATCH 10/56] fs/block_dev: " Pieter Smith
2014-11-13 21:22 ` [PATCH 11/56] fs/bfs: " Pieter Smith
2014-11-13 21:22 ` [PATCH 12/56] fs/btrfs: " Pieter Smith
2014-11-13 21:22 ` [PATCH 13/56] fs/ceph: " Pieter Smith
2014-11-13 21:22 ` Pieter Smith
2014-11-13 21:22 ` [PATCH 14/56] fs/cifs: " Pieter Smith
2014-11-13 21:22 ` [PATCH 15/56] fs/coda: " Pieter Smith
2014-11-13 21:22 ` [PATCH 16/56] fs/encryptfs: " Pieter Smith
2014-11-13 21:22 ` Pieter Smith
2014-11-13 21:22 ` [PATCH 17/56] fs/exofs: " Pieter Smith
2014-11-16 9:02 ` [osd-dev] " Boaz Harrosh
2014-11-13 21:22 ` [PATCH 18/56] fs/ext2: " Pieter Smith
2014-11-13 21:22 ` [PATCH 19/56] fs/ext3: " Pieter Smith
2014-11-13 21:22 ` [PATCH 20/56] fs/ext4: " Pieter Smith
2014-11-14 0:28 ` josh
2014-11-13 21:22 ` [PATCH 21/56] fs/f2fs: " Pieter Smith
2014-11-13 21:22 ` Pieter Smith
2014-11-13 21:22 ` [PATCH 22/56] fs/fat: " Pieter Smith
2014-11-13 21:23 ` [PATCH 23/56] fs/fuse: " Pieter Smith
2014-11-13 21:23 ` [Cluster-devel] [PATCH 24/56] fs/gfs2: " Pieter Smith
2014-11-13 21:23 ` Pieter Smith
2014-11-13 21:23 ` [PATCH 25/56] fs/hfs: " Pieter Smith
2014-11-13 21:23 ` [PATCH 26/56] fs/hfsplus: " Pieter Smith
2014-11-13 21:23 ` [PATCH 27/56] fs/hostfs: " Pieter Smith
2014-11-13 21:23 ` Pieter Smith
2014-11-13 21:23 ` [PATCH 28/56] fs/hpfs: " Pieter Smith
2014-11-13 21:23 ` [PATCH 29/56] fs/jffs2: " Pieter Smith
2014-11-13 21:23 ` Pieter Smith
2014-11-13 21:23 ` [PATCH 30/56] fs/jfs: " Pieter Smith
2014-11-13 21:23 ` [PATCH 31/56] fs/minix: " Pieter Smith
2014-11-13 21:23 ` [PATCH 32/56] fs/nfs: " Pieter Smith
2014-11-13 21:23 ` [PATCH 33/56] fs/nfsd: " Pieter Smith
2014-11-13 21:23 ` [PATCH 34/56] fs/nilfs2: " Pieter Smith
2014-11-13 21:23 ` Pieter Smith
2014-11-13 21:23 ` [PATCH 35/56] fs/ntfs: " Pieter Smith
2014-11-13 21:23 ` [Ocfs2-devel] [PATCH 36/56] fs/ocfs2: " Pieter Smith
2014-11-13 21:23 ` Pieter Smith
2014-11-13 21:23 ` [PATCH 37/56] fs/omfs: " Pieter Smith
2014-11-13 21:23 ` [PATCH 38/56] fs/ramfs: " Pieter Smith
2014-11-13 21:23 ` [PATCH 39/56] fs/reiserfs: " Pieter Smith
2014-11-13 21:23 ` Pieter Smith
2014-11-13 21:23 ` [PATCH 40/56] fs/romfs: " Pieter Smith
2014-11-13 21:23 ` [PATCH 41/56] fs/sysv: " Pieter Smith
2014-11-13 21:23 ` [PATCH 42/56] fs/ubifs: " Pieter Smith
2014-11-13 21:23 ` Pieter Smith
2014-11-13 21:23 ` [PATCH 43/56] fs/udf: " Pieter Smith
2014-11-13 21:23 ` [PATCH 44/56] fs/ufs: " Pieter Smith
2014-11-13 21:23 ` [PATCH 45/56] fs/xfs: " Pieter Smith
2014-11-13 21:23 ` [PATCH 46/56] kernel/relay: " Pieter Smith
2014-11-13 21:23 ` [PATCH 47/56] kenel/trace: " Pieter Smith
2014-11-17 20:33 ` Steven Rostedt
2014-11-13 21:23 ` [PATCH 48/56] mm/shmem: " Pieter Smith
2014-11-13 21:23 ` Pieter Smith
2014-11-13 21:23 ` [PATCH 49/56] net/socket: " Pieter Smith
2014-11-13 21:23 ` [PATCH 50/56] fs/read_write: " Pieter Smith
2014-11-13 21:23 ` [PATCH 51/56] drivers/char/mem: " Pieter Smith
2014-11-13 22:09 ` Greg Kroah-Hartman
[not found] ` <CAPho-_JJGy0cwBVfWKL1Gt9ZQZM+Odo7W05bKQ2JLO+TM-ABJA@mail.gmail.com>
2014-11-13 22:31 ` josh
2014-11-13 23:34 ` Greg Kroah-Hartman
2014-11-14 0:19 ` josh
2014-11-14 3:27 ` Greg Kroah-Hartman
[not found] ` <CAPho-_KUc-+c=X6xtLTD2F-o4qi+YpYdnw6F1cx4kniezfs7aw@mail.gmail.com>
2014-11-14 23:25 ` josh
[not found] ` <CAPho-_LCj7bxJ2EvuZtBZXD1buH1V+nKEiobTJUATYmGYVRWcA@mail.gmail.com>
2014-11-16 18:20 ` Josh Triplett
[not found] ` <CAPho-_+ZXHYGB9-d19NR9u5tcOpN1Ytg80NZgvScF=YQ-6SdNA@mail.gmail.com>
2014-11-18 22:42 ` josh
2014-11-13 21:23 ` [PATCH 52/56] drivers/char/virtio: " Pieter Smith
2014-11-13 21:23 ` Pieter Smith
2014-11-13 22:09 ` Greg Kroah-Hartman [this message]
2014-11-13 22:09 ` Greg Kroah-Hartman
2014-11-13 21:23 ` [PATCH 53/56] net/ipv6: " Pieter Smith
2014-11-13 21:23 ` [PATCH 54/56] net/ipv4: " Pieter Smith
2014-11-13 21:23 ` [PATCH 55/56] net/core: " Pieter Smith
2014-11-13 21:23 ` [PATCH 56/56] fs/splice: full support for " Pieter Smith
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20141113220949.GC546@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=amit.shah@redhat.com \
--cc=arnd@arndb.de \
--cc=josh@joshtriplett.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pieter@boesman.nl \
--cc=virtualization@lists.linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.