Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp10509559imu; Thu, 6 Dec 2018 02:17:28 -0800 (PST) X-Google-Smtp-Source: AFSGD/XVOl4zHcJ3Lcyon/duNfugFbU9ZeWA4EUBOmNmrG782n/tuy+CRlx+6achQBWeeMor9btb X-Received: by 2002:a63:181c:: with SMTP id y28mr22901707pgl.75.1544091448342; Thu, 06 Dec 2018 02:17:28 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1544091448; cv=none; d=google.com; s=arc-20160816; b=pGpxMO4oftXWUDmBwsq99zpTDo3zGrWdStuaZwzG8H5Z+EUz79OdK8qsckd8wcyIKz 8wf2b3zGMp61JLuTYX7tuP2v+5qhP5PvjLbka2eV9RFkaPPeoWzTazcZsgonq59ZpBNR P45V3Yszt0n65xq852zmFRsIo8AXIyJ4TBIo5PhIQlR3mJPWUjy82TZ6EVhrl49Zffq2 lOAAbuibNZyAGA0JPAnLuYV03t+Uw/Jt793ozd7G8v4Fs47/98/udE+nSfSQ53Gu1SlL ufQFc9KHWsiviYW0AIeUw3LqeC/YO/DSBdRhy6CC/Y8V81jGlCgIsrbWwuk4ywqwTLTS fXOQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=iqKzkJpGavy+5uAJ2D5Im0iftTI2Kxvs32M6gUsgBFs=; b=llkVpNwOl0ZdRmBXo9ogDt9PPvUxFhqAriBuxKuWkshFbkb2TcUCbQTxidfDBdBpQS tYUkHte7U7+VwlTmfq0oFdIlf6W9NQRHpW+wQcBeg2iOkz6/bIhVwVPECObn7fTJrHfg /0V0tOHG9jgSa2RdQSmYQB+kC1NoVuZAoTsWMHLMW+hR5Jc0SnyIxO6411oNY8ivwolv KHnoGiFdwi6Yesl/KGUdauARWXKSNQD/KmtsQbJLLTdU8zIfoOFMI2106/7lDAgMslFo ltuj4VRZwzJp7f9hCN8rdy5I96uWTQCVS4KGx4ZRsG0Ko+El3Ac27zyuOnKjZkUM2+7T uaVA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v9si23664172pfl.45.2018.12.06.02.17.12; Thu, 06 Dec 2018 02:17:28 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729349AbeLFKQC (ORCPT + 99 others); Thu, 6 Dec 2018 05:16:02 -0500 Received: from mx2.suse.de ([195.135.220.15]:37078 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727806AbeLFKP6 (ORCPT ); Thu, 6 Dec 2018 05:15:58 -0500 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id 55BE6AE3C; Thu, 6 Dec 2018 10:15:56 +0000 (UTC) Date: Thu, 6 Dec 2018 11:15:55 +0100 From: Petr Mladek To: Miroslav Benes Cc: Joe Lawrence , Jiri Kosina , Josh Poimboeuf , Jason Baron , Evgenii Shatokhin , live-patching@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v14 00/11] livepatch: Atomic replace feature Message-ID: <20181206101555.cydbzb7smubzsavn@pathway.suse.cz> References: <20181129094431.7801-1-pmladek@suse.com> <176e592d-6e9c-8e1c-abe6-d07ba9b10d54@redhat.com> <20181206075404.rv4qyw2u5auvfiuz@pathway.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170421 (1.8.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu 2018-12-06 10:32:00, Miroslav Benes wrote: > > > > I don't have many code comments as the changes appear to safely and > > > correctly do what the say. (We are at v14 after all :) I mainly > > > compared the text and comments to the implementation and noted typos > > > (marked by substitution s/old/new) and awkward wordings (marked by > > > "re-wording suggestion"). That said, I ack'd each patch as I wouldn't > > > want these to hold up the patchset. > > > > Thanks a lot both you and Miroslav for the review. > > > > I'll give it some more days before I prepare v15. I wonder if Josh > > could find some cycle to look at it at least from the top level. > > For what is worth, I'm fine with all the changes Joe proposed and you can > preserve my acks there. We need enough time for bikeshedding about comments and documentation. I need some time to recover. And I still hope that Josh can give it a spin. Best Regards, Petr