All of lore.kernel.org
 help / color / mirror / Atom feed
From: Josh Triplett <josh@joshtriplett.org>
To: Pieter Smith <pieter@boesman.nl>
Cc: Alexander Duyck <alexander.h.duyck@intel.com>,
	Alexander Viro <viro@zeniv.linux.org.uk>,
	Alexei Starovoitov <ast@plumgrid.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Bertrand Jacquin <beber@meleeweb.net>,
	Catalina Mocanu <catalina.mocanu@gmail.com>,
	Daniel Borkmann <dborkman@redhat.com>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	Fabian Frederick <fabf@skynet.be>,
	"open list:FUSE: FILESYSTEM..."
	<fuse-devel@lists.sourceforge.net>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Hugh Dickins <hughd@google.com>,
	Iulia Manda <iulia.manda21@gmail.com>,
	Jan Beulich <JBeulich@suse.com>,
	"J. Bruce Fields" <bfields@fieldses.org>,
	Jeff Layton <jlayton@poochiereds.net>,
	"open list:ABI/API" <linux-api@vger.kernel.org>,
	linux-fsdevel@vger.kernel.org,
	open list <linux-kernel@vger.kernel.org>open
Subject: Re: [PATCH v4 0/7] kernel tinification: optionally compile out splice family of syscalls (splice, vmsplice, tee and sendfile)
Date: Mon, 24 Nov 2014 16:52:21 -0800	[thread overview]
Message-ID: <20141125005221.GB21653@thin> (raw)
In-Reply-To: <1416870079-15254-1-git-send-email-pieter@boesman.nl>

On Tue, Nov 25, 2014 at 12:00:59AM +0100, Pieter Smith wrote:
> REPO: https://github.com/smipi1/linux-tinification.git
> 
> BRANCH: tiny/config-syscall-splice
> 
> BACKGROUND: This patch-set forms part of the Linux Kernel Tinification effort (
>   https://tiny.wiki.kernel.org/).
> 
> GOAL: Support compiling out the splice family of syscalls (splice, vmsplice,
>   tee and sendfile) along with all supporting infrastructure if not needed.
>   Many embedded systems will not need the splice-family syscalls. Omitting them
>   saves space.
> 
> HISTORY:
>   PATCH v4:
>     - Drops __splice_p()
>     - Let nfsd fall back to non-splice support when splice is compiled out
>     - Style fixes
[...]
> RESULTS: A tinyconfig bloat-o-meter score for the entire patch-set:
> 
> add/remove: 0/41 grow/shrink: 5/7 up/down: 23/-8422 (-8399)

I replied to one patch with a minor nit in the commit message.  Other
than that, I don't see any obvious issues with this.

- Josh Triplett

WARNING: multiple messages have this Message-ID (diff)
From: Josh Triplett <josh@joshtriplett.org>
To: Pieter Smith <pieter@boesman.nl>
Cc: "Alexander Duyck" <alexander.h.duyck@intel.com>,
	"Alexander Viro" <viro@zeniv.linux.org.uk>,
	"Alexei Starovoitov" <ast@plumgrid.com>,
	"Andrew Morton" <akpm@linux-foundation.org>,
	"Bertrand Jacquin" <beber@meleeweb.net>,
	"Catalina Mocanu" <catalina.mocanu@gmail.com>,
	"Daniel Borkmann" <dborkman@redhat.com>,
	"David S. Miller" <davem@davemloft.net>,
	"Eric Dumazet" <edumazet@google.com>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	"Fabian Frederick" <fabf@skynet.be>,
	"open list:FUSE: FILESYSTEM..."
	<fuse-devel@lists.sourceforge.net>,
	"Geert Uytterhoeven" <geert@linux-m68k.org>,
	"Hugh Dickins" <hughd@google.com>,
	"Iulia Manda" <iulia.manda21@gmail.com>,
	"Jan Beulich" <JBeulich@suse.com>,
	"J. Bruce Fields" <bfields@fieldses.org>,
	"Jeff Layton" <jlayton@poochiereds.net>,
	"open list:ABI/API" <linux-api@vger.kernel.org>,
	linux-fsdevel@vger.kernel.org,
	"open list" <linux-kernel@vger.kernel.org>,
	"open list:KERNEL NFSD, SUNR..." <linux-nfs@vger.kernel.org>,
	"Luis R. Rodriguez" <mcgrof@suse.com>,
	"Matt Turner" <mattst88@gmail.com>,
	"Mel Gorman" <mgorman@suse.de>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Miklos Szeredi" <miklos@szeredi.hu>,
	"open list:NETWORKING [GENERAL]" <netdev@vger.kernel.org>,
	"Oleg Nesterov" <oleg@redhat.com>,
	"Paul Durrant" <Paul.Durrant@citrix.com>,
	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
	"Peter Foley" <pefoley2@pefoley.com>,
	"Thomas Graf" <tgraf@suug.ch>,
	"Tom Herbert" <therbert@google.com>,
	"Trond Myklebust" <trond.myklebust@primarydata.com>,
	"Willem de Bruijn" <willemb@google.com>,
	"Xiao Guangrong" <xiaoguangrong@linux.vnet.ibm.com>,
	蔡正龙 <zhenglong.cai@cs2c.com.cn>
Subject: Re: [PATCH v4 0/7] kernel tinification: optionally compile out splice family of syscalls (splice, vmsplice, tee and sendfile)
Date: Mon, 24 Nov 2014 16:52:21 -0800	[thread overview]
Message-ID: <20141125005221.GB21653@thin> (raw)
In-Reply-To: <1416870079-15254-1-git-send-email-pieter@boesman.nl>

On Tue, Nov 25, 2014 at 12:00:59AM +0100, Pieter Smith wrote:
> REPO: https://github.com/smipi1/linux-tinification.git
> 
> BRANCH: tiny/config-syscall-splice
> 
> BACKGROUND: This patch-set forms part of the Linux Kernel Tinification effort (
>   https://tiny.wiki.kernel.org/).
> 
> GOAL: Support compiling out the splice family of syscalls (splice, vmsplice,
>   tee and sendfile) along with all supporting infrastructure if not needed.
>   Many embedded systems will not need the splice-family syscalls. Omitting them
>   saves space.
> 
> HISTORY:
>   PATCH v4:
>     - Drops __splice_p()
>     - Let nfsd fall back to non-splice support when splice is compiled out
>     - Style fixes
[...]
> RESULTS: A tinyconfig bloat-o-meter score for the entire patch-set:
> 
> add/remove: 0/41 grow/shrink: 5/7 up/down: 23/-8422 (-8399)

I replied to one patch with a minor nit in the commit message.  Other
than that, I don't see any obvious issues with this.

- Josh Triplett

WARNING: multiple messages have this Message-ID (diff)
From: Josh Triplett <josh@joshtriplett.org>
To: Pieter Smith <pieter@boesman.nl>
Cc: Alexander Duyck <alexander.h.duyck@intel.com>,
	Alexander Viro <viro@zeniv.linux.org.uk>,
	Alexei Starovoitov <ast@plumgrid.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Bertrand Jacquin <beber@meleeweb.net>,
	Catalina Mocanu <catalina.mocanu@gmail.com>,
	Daniel Borkmann <dborkman@redhat.com>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	Fabian Frederick <fabf@skynet.be>,
	"open list:FUSE: FILESYSTEM..."
	<fuse-devel@lists.sourceforge.net>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Hugh Dickins <hughd@google.com>,
	Iulia Manda <iulia.manda21@gmail.com>,
	Jan Beulich <JBeulich@suse.com>,
	"J. Bruce Fields" <bfields@fieldses.org>,
	Jeff Layton <jlayton@poochiereds.net>,
	"open list:ABI/API" <linux-api@vger.kernel.org>,
	linux-fsdevel@vger.kernel.org,
	open list <linux-kernel@vger.kernel.org>,
	"open
Subject: Re: [PATCH v4 0/7] kernel tinification: optionally compile out splice family of syscalls (splice, vmsplice, tee and sendfile)
Date: Mon, 24 Nov 2014 16:52:21 -0800	[thread overview]
Message-ID: <20141125005221.GB21653@thin> (raw)
In-Reply-To: <1416870079-15254-1-git-send-email-pieter@boesman.nl>

On Tue, Nov 25, 2014 at 12:00:59AM +0100, Pieter Smith wrote:
> REPO: https://github.com/smipi1/linux-tinification.git
> 
> BRANCH: tiny/config-syscall-splice
> 
> BACKGROUND: This patch-set forms part of the Linux Kernel Tinification effort (
>   https://tiny.wiki.kernel.org/).
> 
> GOAL: Support compiling out the splice family of syscalls (splice, vmsplice,
>   tee and sendfile) along with all supporting infrastructure if not needed.
>   Many embedded systems will not need the splice-family syscalls. Omitting them
>   saves space.
> 
> HISTORY:
>   PATCH v4:
>     - Drops __splice_p()
>     - Let nfsd fall back to non-splice support when splice is compiled out
>     - Style fixes
[...]
> RESULTS: A tinyconfig bloat-o-meter score for the entire patch-set:
> 
> add/remove: 0/41 grow/shrink: 5/7 up/down: 23/-8422 (-8399)

I replied to one patch with a minor nit in the commit message.  Other
than that, I don't see any obvious issues with this.

- Josh Triplett

  parent reply	other threads:[~2014-11-25  0:52 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-24 23:00 [PATCH v4 0/7] kernel tinification: optionally compile out splice family of syscalls (splice, vmsplice, tee and sendfile) Pieter Smith
2014-11-24 23:00 ` Pieter Smith
2014-11-24 23:00 ` Pieter Smith
2014-11-24 23:01 ` [PATCH v4 1/7] fs: move sendfile syscall into fs/splice Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-24 23:01 ` [PATCH v4 2/7] fs: moved kernel_write to fs/read_write Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-24 23:01 ` [PATCH v4 3/7] fs/splice: support compiling out splice-family syscalls Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-25  0:49   ` Josh Triplett
2014-11-25  0:49     ` Josh Triplett
2014-11-25  0:49     ` Josh Triplett
2014-11-24 23:01 ` [PATCH v4 4/7] fs/fuse: support compiling out splice Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-24 23:01 ` [PATCH v4 5/7] fs/nfsd: " Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-24 23:01   ` Pieter Smith
     [not found] ` <1416870079-15254-1-git-send-email-pieter-qeJ+1H9vRZbz+pZb47iToQ@public.gmane.org>
2014-11-24 23:01   ` [PATCH v4 6/7] net/core: " Pieter Smith
2014-11-24 23:01     ` Pieter Smith
2014-11-24 23:01 ` [PATCH v4 7/7] fs/splice: full support for " Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-24 23:01   ` Pieter Smith
2014-11-25  0:52 ` Josh Triplett [this message]
2014-11-25  0:52   ` [PATCH v4 0/7] kernel tinification: optionally compile out splice family of syscalls (splice, vmsplice, tee and sendfile) Josh Triplett
2014-11-25  0:52   ` Josh Triplett
     [not found] ` <5474ABB6.3030400@infradead.org>
     [not found]   ` <5474ABB6.3030400-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
2014-11-25 17:13     ` David Miller
2014-11-25 17:13       ` David Miller
2014-11-25 18:10       ` Paul E. McKenney
2014-11-25 18:10         ` Paul E. McKenney
2014-11-25 18:24         ` David Miller
2014-11-25 18:24           ` David Miller
     [not found]           ` <20141125.132445.152609149279137368.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
2014-11-25 18:58             ` Theodore Ts'o
2014-11-25 18:58               ` Theodore Ts'o
     [not found]               ` <20141125185806.GA28116-AKGzg7BKzIDYtjvyW6yDsg@public.gmane.org>
2014-11-25 19:05                 ` David Miller
2014-11-25 19:05                   ` David Miller
2014-11-25 18:53       ` josh
2014-11-25 18:53         ` josh
2014-11-25 19:04         ` David Miller
2014-11-25 19:04           ` David Miller
     [not found]           ` <20141125.140441.401150380839514113.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
2014-11-25 19:16             ` Eric W. Biederman
2014-11-25 19:16               ` Eric W. Biederman
     [not found]               ` <87egsr9jkz.fsf-JOvCrm2gF+uungPnsOpG7nhyD016LWXt@public.gmane.org>
2014-11-25 19:27                 ` David Miller
2014-11-25 19:27                   ` David Miller
     [not found]                   ` <20141125.142741.1620673255148724338.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
2014-11-25 20:01                     ` Eric W. Biederman
2014-11-25 20:01                       ` Eric W. Biederman
2014-11-25 20:11             ` Pieter Smith
2014-11-25 20:11               ` Pieter Smith
2014-11-25 20:11               ` Pieter Smith
2014-11-26 12:19           ` One Thousand Gnomes
2014-11-26 12:19             ` One Thousand Gnomes
2014-11-25 19:00     ` josh-iaAMLnmF4UmaiuxdJuQwMA
2014-11-25 22:08     ` josh-iaAMLnmF4UmaiuxdJuQwMA
2014-11-25 22:08       ` josh-iaAMLnmF4UmaiuxdJuQwMA
2014-11-25 22:08       ` josh

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=20141125005221.GB21653@thin \
    --to=josh@joshtriplett.org \
    --cc=JBeulich@suse.com \
    --cc=akpm@linux-foundation.org \
    --cc=alexander.h.duyck@intel.com \
    --cc=ast@plumgrid.com \
    --cc=beber@meleeweb.net \
    --cc=bfields@fieldses.org \
    --cc=catalina.mocanu@gmail.com \
    --cc=davem@davemloft.net \
    --cc=dborkman@redhat.com \
    --cc=ebiederm@xmission.com \
    --cc=edumazet@google.com \
    --cc=fabf@skynet.be \
    --cc=fuse-devel@lists.sourceforge.net \
    --cc=geert@linux-m68k.org \
    --cc=hughd@google.com \
    --cc=iulia.manda21@gmail.com \
    --cc=jlayton@poochiereds.net \
    --cc=linux-api@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pieter@boesman.nl \
    --cc=viro@zeniv.linux.org.uk \
    /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.