[{"id":1772218,"web_url":"http://patchwork.ozlabs.org/comment/1772218/","msgid":"<20170920205642.GA20023@infradead.org>","list_archive_url":null,"date":"2017-09-20T20:56:42","subject":"Re: [PATCH v3 14/31] vxfs: Define usercopy region in vxfs_inode slab\n\tcache","submitter":{"id":178,"url":"http://patchwork.ozlabs.org/api/people/178/","name":"Christoph Hellwig","email":"hch@infradead.org"},"content":"Hi Kees,\n\nI've only got this single email from you, which on it's own doesn't\ncompile and seems to be part of a 31 patch series.\n\nSo as-is NAK, doesn't work.\n\nPlease make sure to always send every patch in a series to every\ndeveloper you want to include.","headers":{"Return-Path":"","X-Original-To":"patchwork-incoming@ozlabs.org","Delivered-To":"patchwork-incoming@ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netdev-owner@vger.kernel.org;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=infradead.org header.i=@infradead.org\n\theader.b=\"unCs1PmT\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyBrG0xMxz9s83\n\tfor ;\n\tThu, 21 Sep 2017 06:57:02 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1752163AbdITU4r (ORCPT );\n\tWed, 20 Sep 2017 16:56:47 -0400","from bombadil.infradead.org ([65.50.211.133]:38412 \"EHLO\n\tbombadil.infradead.org\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751465AbdITU4n (ORCPT\n\t); Wed, 20 Sep 2017 16:56:43 -0400","from hch by bombadil.infradead.org with local (Exim 4.87 #1 (Red\n\tHat Linux)) id 1dum30-0005tH-5K; Wed, 20 Sep 2017 20:56:42 +0000"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=infradead.org; s=bombadil.20170209;\n\th=In-Reply-To:Content-Type:MIME-Version\n\t:References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To:\n\tContent-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:\n\tList-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive;\n\tbh=Hi7l2K2nB7SM5K0Ptcmh/BtIGrxsYdapABDthMqDnTw=;\n\tb=unCs1PmTCYkujHmkdnr0kPqmQ\n\tpx+8FWuua5Kth6/e5zTzwFzeAMPx9fYvFoAnuT1pz7Hryj6mhgGWQstYgMB+GJLPh8QYrD0CfOKZZ\n\tdjGph17JtSGw4kBHyX3Y9nwnENCdGFwdIUI6hVtZzs10AdPXAkRTOo4Q28JRy4nQh4u3bHc+J1YYi\n\tpFR4KG1DqAbqOWmNck5O4kE654E55/YNgmkaaO5i+wxuxYJfzS1D6UPFp6gsowMFk2U+Dv+sAlEYD\n\t901Gcj8jsyNBMCxyD3I3AdOO1rCwIYWJ+bFyUXAhk3VkIxt2htfdhBWdPhtlEkiy7j+9ctkPF8VxZ\n\tdoaXR3/8g==;","Date":"Wed, 20 Sep 2017 13:56:42 -0700","From":"Christoph Hellwig ","To":"Kees Cook ","Cc":"linux-kernel@vger.kernel.org, David Windsor ,\n\tChristoph Hellwig ,\n\tlinux-fsdevel@vger.kernel.org, netdev@vger.kernel.org,\n\tlinux-mm@kvack.org, kernel-hardening@lists.openwall.com","Subject":"Re: [PATCH v3 14/31] vxfs: Define usercopy region in vxfs_inode slab\n\tcache","Message-ID":"<20170920205642.GA20023@infradead.org>","References":"<1505940337-79069-1-git-send-email-keescook@chromium.org>\n\t<1505940337-79069-15-git-send-email-keescook@chromium.org>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<1505940337-79069-15-git-send-email-keescook@chromium.org>","User-Agent":"Mutt/1.8.3 (2017-05-23)","X-SRS-Rewrite":"SMTP reverse-path rewritten from by\n\tbombadil.infradead.org. See http://www.infradead.org/rpr.html","Sender":"netdev-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"netdev@vger.kernel.org"}},{"id":1772251,"web_url":"http://patchwork.ozlabs.org/comment/1772251/","msgid":"","list_archive_url":null,"date":"2017-09-20T21:21:45","subject":"Re: [PATCH v3 14/31] vxfs: Define usercopy region in vxfs_inode\n\tslab cache","submitter":{"id":10641,"url":"http://patchwork.ozlabs.org/api/people/10641/","name":"Kees Cook","email":"keescook@chromium.org"},"content":"On Wed, Sep 20, 2017 at 1:56 PM, Christoph Hellwig wrote:\n> Hi Kees,\n>\n> I've only got this single email from you, which on it's own doesn't\n> compile and seems to be part of a 31 patch series.\n>\n> So as-is NAK, doesn't work.\n>\n> Please make sure to always send every patch in a series to every\n> developer you want to include.\n\nThis is why I included several other lists on the full CC (am I\nunlucky enough to have you not subscribed to any of them?). Adding a\nCC for everyone can result in a huge CC list, especially for the\nforth-coming 300-patch timer_list series. ;)\n\nDo you want me to resend the full series to you, or would you prefer\nsomething else like a patchwork bundle? (I'll explicitly add you to CC\nfor any future versions, though.)\n\n-Kees","headers":{"Return-Path":"","X-Original-To":"patchwork-incoming@ozlabs.org","Delivered-To":"patchwork-incoming@ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netdev-owner@vger.kernel.org;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"qcIzO3aP\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"GSrQmVBi\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyCP73NVhz9s8J\n\tfor ;\n\tThu, 21 Sep 2017 07:22:03 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751583AbdITVVt (ORCPT );\n\tWed, 20 Sep 2017 17:21:49 -0400","from mail-it0-f49.google.com ([209.85.214.49]:51789 \"EHLO\n\tmail-it0-f49.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751387AbdITVVr (ORCPT\n\t); Wed, 20 Sep 2017 17:21:47 -0400","by mail-it0-f49.google.com with SMTP id o200so4487419itg.0\n\tfor ; Wed, 20 Sep 2017 14:21:47 -0700 (PDT)","by 10.107.178.131 with HTTP; Wed, 20 Sep 2017 14:21:45 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=google.com; s=20161025;\n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=866yGR5JCLmFRJ+R/igRsmCDeN+ynz+kQQjkfWFgxCU=;\n\tb=qcIzO3aPamOw2XAUN4mu42jvrflerAv7uUKKn95VCrCyr4Hum38cFdZEqncjuCVdlV\n\tCp9Nn+k8VqfLzk/vQjf8UeZ0SH8QY7IZcOUOjUoBV+jMrrgI6DamZ9IGf1Fx83033c19\n\tKxd3SDhf7F2OS103UwWy/zuR2F0rDIAkfxf57O1+e0GCcSoLchY6rtNVBwtHeX1aXca+\n\trj+Hn8/d41VLjrXN3Ow+JHCVJMnG2mPgnY666BIZ4E8H9Ew5k7+zddjYl4mQ86JU5nbM\n\tnAwf+UrU7uShj0cMyxoVZafQcwe4ssa7Jro2OBMT3BjfFsZ6aTmLXyzE5qIz0gGgGtOf\n\tpmYw==","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=chromium.org; s=google;\n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=866yGR5JCLmFRJ+R/igRsmCDeN+ynz+kQQjkfWFgxCU=;\n\tb=GSrQmVBi+SBrz/AcS/hQs3iqpAWH6+LxGxIfgDxHlcRZJrujzMHcmXe/twJ90fCJbT\n\tt3FY61VnPn/xPBP78kB8PkS1HaTUCOI03DwQW2Ls03FtWlyK+qwYFqdCXVzTW9yUvpvE\n\trYNotdrhQiTycIfg9MtI2kUjuKRF5gbsqHW7Y="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:sender:in-reply-to:references:from\n\t:date:message-id:subject:to:cc;\n\tbh=866yGR5JCLmFRJ+R/igRsmCDeN+ynz+kQQjkfWFgxCU=;\n\tb=GEUa46eZ9D/Txh5WWA2akS7E6RJGDXhtV/sZBZV0Lk2/t7j8ZHOqlyohudO1Wwz3IE\n\tbT80ANjqvhnSqmREmvtG0dilleIJ9toIVj3Envv5EnOOnJdK660veNrC5M3pKvtLyQR4\n\tgi3xee3b5mWf4YkPqIy4oFRhrdw2ztTRXDQUShEwb5CDZqpnqwdhInvGcMVpqKPr1Eg1\n\tTD/UI1HzFu/W816C1FBVRn+kphNvWp0e+Jtof/ythNjZzTPrx3fQ0E3RMERzzn3hppi/\n\t+jWUgylt6YYfvrxNqEST9kjNjDbu1bBgKmmjAJyJSP4Tn2qtAfsI21JWuOfKXB1kpdM+\n\tUBAQ==","X-Gm-Message-State":"AHPjjUhJbU+OUopFCEKdpviFnIhqUVv6AYV+kvRddXKppuo5ylzNTcwU\n\tRqnfGf5hXEmVGiAYpPZJezQjpeb13mCrcv+VGo6Gjg==","X-Google-Smtp-Source":"AOwi7QB0feWjvuTTZZAUciPbABu0aVI2vzZ24MxSfLaDqVc4mEXItE4jfsaBFtDVxw+AuKtzxZiR/8TdVbYzeHWG8jY=","X-Received":"by 10.36.141.67 with SMTP id w64mr4848820itd.8.1505942506524;\n\tWed, 20 Sep 2017 14:21:46 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170920205642.GA20023@infradead.org>","References":"<1505940337-79069-1-git-send-email-keescook@chromium.org>\n\t<1505940337-79069-15-git-send-email-keescook@chromium.org>\n\t<20170920205642.GA20023@infradead.org>","From":"Kees Cook ","Date":"Wed, 20 Sep 2017 14:21:45 -0700","X-Google-Sender-Auth":"lt7lUxvq5FovLvFgrxeMzCrJGnY","Message-ID":"","Subject":"Re: [PATCH v3 14/31] vxfs: Define usercopy region in vxfs_inode\n\tslab cache","To":"Christoph Hellwig ","Cc":"LKML , David Windsor ,\n\t\"linux-fsdevel@vger.kernel.org\" ,\n\tNetwork Development ,\n\tLinux-MM , \"kernel-hardening@lists.openwall.com\" \n\t","Content-Type":"text/plain; charset=\"UTF-8\"","Sender":"netdev-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"netdev@vger.kernel.org"}},{"id":1772312,"web_url":"http://patchwork.ozlabs.org/comment/1772312/","msgid":"<20170920232221.GA18311@infradead.org>","list_archive_url":null,"date":"2017-09-20T23:22:21","subject":"Re: [PATCH v3 14/31] vxfs: Define usercopy region in vxfs_inode slab\n\tcache","submitter":{"id":178,"url":"http://patchwork.ozlabs.org/api/people/178/","name":"Christoph Hellwig","email":"hch@infradead.org"},"content":"On Wed, Sep 20, 2017 at 02:21:45PM -0700, Kees Cook wrote:\n> This is why I included several other lists on the full CC (am I\n> unlucky enough to have you not subscribed to any of them?). Adding a\n> CC for everyone can result in a huge CC list, especially for the\n> forth-coming 300-patch timer_list series. ;)\n\nIf you think the lists are enough to review changes include only\nthe lists, but don't add CCs for individual patches, that's what\nI usually do for cleanups that touch a lot of drivers, but don't\nreally change actual logic in ever little driver touched.\n\n> Do you want me to resend the full series to you, or would you prefer\n> something else like a patchwork bundle? (I'll explicitly add you to CC\n> for any future versions, though.)\n\nI'm fine with not being Cced at all if there isn't anything requiring\nmy urgent personal attention. It's up to you whom you want to Cc,\nbut my preference is generally for rather less than more people, and\nrather more than less mailing lists.\n\nBut the important bit is to Cc a person or mailinglist either on\nall patches or on none, otherwise a good review isn't possible.","headers":{"Return-Path":"","X-Original-To":"patchwork-incoming@ozlabs.org","Delivered-To":"patchwork-incoming@ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netdev-owner@vger.kernel.org;\n\treceiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=infradead.org header.i=@infradead.org\n\theader.b=\"tpv8kwEh\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyG4B6Tr2z9s0Z\n\tfor ;\n\tThu, 21 Sep 2017 09:22:34 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751905AbdITXWY (ORCPT );\n\tWed, 20 Sep 2017 19:22:24 -0400","from bombadil.infradead.org ([65.50.211.133]:37941 \"EHLO\n\tbombadil.infradead.org\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751813AbdITXWX (ORCPT\n\t); Wed, 20 Sep 2017 19:22:23 -0400","from hch by bombadil.infradead.org with local (Exim 4.87 #1 (Red\n\tHat Linux)) id 1duoJy-0006Vm-0E; Wed, 20 Sep 2017 23:22:22 +0000"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=infradead.org; s=bombadil.20170209;\n\th=In-Reply-To:Content-Type:MIME-Version\n\t:References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To:\n\tContent-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:\n\tList-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive;\n\tbh=JV6i0snS3xlUnJixggRxblbo3X0yxpSk+AwFukukPVI=;\n\tb=tpv8kwEhZQFV5Frd3a3Zf2bsG\n\tZNtaYEwcVUP6eHvKjhACcti6JuQlE4UcNck0StFoq2xc4gmWPc1FZhZo8Erpr0vW3YdK7ZOBvh6S2\n\t4LgqY+a/hSgnAnx7obALITpMHfR0Sx/5nYdWovWQ5BRKWukBtS8jTJlw+F96KOpcxngrQfu+lZYhg\n\t+JmLlJbfzyDCQF54Ey+5Y5OaiWR5g1VWYjnxqhPI5vcPlo3BOgBqGupdmjo+qnvPDFrsgxKCMql//\n\tK2TQzMac6U8mHEpPRL2vVnxlxZzHMmkI0bw7jWHGPlUuHdAk8x2w+wKtXdhS4PusdtZXH8ZtLhFGa\n\teXAppCd3A==;","Date":"Wed, 20 Sep 2017 16:22:21 -0700","From":"Christoph Hellwig ","To":"Kees Cook ","Cc":"Christoph Hellwig ,\n\tLKML , David Windsor ,\n\t\"linux-fsdevel@vger.kernel.org\" ,\n\tNetwork Development ,\n\tLinux-MM , \"kernel-hardening@lists.openwall.com\" \n\t","Subject":"Re: [PATCH v3 14/31] vxfs: Define usercopy region in vxfs_inode slab\n\tcache","Message-ID":"<20170920232221.GA18311@infradead.org>","References":"<1505940337-79069-1-git-send-email-keescook@chromium.org>\n\t<1505940337-79069-15-git-send-email-keescook@chromium.org>\n\t<20170920205642.GA20023@infradead.org>\n\t","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"","User-Agent":"Mutt/1.8.3 (2017-05-23)","X-SRS-Rewrite":"SMTP reverse-path rewritten from by\n\tbombadil.infradead.org. See http://www.infradead.org/rpr.html","Sender":"netdev-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"netdev@vger.kernel.org"}}]