Received: by 10.223.185.116 with SMTP id b49csp4719338wrg; Tue, 27 Feb 2018 01:09:40 -0800 (PST) X-Google-Smtp-Source: AH8x2270nO6AgBdBAAj6Wnx742EyEhYuCtRKxNQcJI+I30X8o3B60x7lq8I7AMeIsFw9bhMWPWZw X-Received: by 10.98.252.22 with SMTP id e22mr13448648pfh.235.1519722580342; Tue, 27 Feb 2018 01:09:40 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1519722580; cv=none; d=google.com; s=arc-20160816; b=jXbcVp7VrSOZIEcJq5deBFUykZWEfdhQbNkTWSgFsga+bWTvHDcCRa3ZUhvff4bTN8 DYrgKSwSdUzModBygYkgD674+rTMRLYFbRn3ludjwEqPKWSLiGk5RhelOXET/IEqmejC +K5j1FOD1NKVZojPt8G1MjANsx5Vp6ux+27r0BWuS7NqQnR8rcvGVjvdfIoJFPlRuUcn VG2/4vE3/kjzbfjnkDNaGbwFrc90WWj9XVK+StVCqJnZYOcw6UDdESUg24+rKDeJLW7Q WCX5Jb7k3WA/XYt1zqnv4+GyUwUbGdKnTW+fY2zF+0+UZOzPkR2osVHKmtGPM0bZXBy4 NlKA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :organization:references:in-reply-to:message-id:subject:cc:to:from :date:dkim-signature:arc-authentication-results; bh=AqnGsX4ng5eju/LmOKHUeXvJu+ci/40GOFAT+8+6pFg=; b=Plwd0KM4/4xQc9xLqbMtOxUoLLoON8Y1sDQ7o3CFjRV7oBvedxjBSaGui2U9EFKR6s Q/pt5h6muIhelzi11Cc8Mag+qjFXleksszTuFy/FafDieP4Y2Q8KmcBCt5KD8Al9ZLdh Cd6xi5AaWgOhF6W//ty0kHOW367f4rP1Ar1Z1zCpzicelHGoPMs0Oynrwrq472gyX7eT Y0gdlouy/oec1YRUxDP5fAaGu38l48w3GOXCNiDxhV+ItsTy2uOb2QLX5vzoRo8/CCI2 1/uY3Y/74PkH7zl88+uq/28Ep5VVHhfPzNLzceubkkvw813aX8+d9so7MsbbqGq2czJu QQGg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=N4GSBnnW; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x189si6745809pgb.216.2018.02.27.01.09.25; Tue, 27 Feb 2018 01:09:40 -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; dkim=pass header.i=@gmail.com header.s=20161025 header.b=N4GSBnnW; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752382AbeB0JIc (ORCPT + 99 others); Tue, 27 Feb 2018 04:08:32 -0500 Received: from mail-pg0-f49.google.com ([74.125.83.49]:33534 "EHLO mail-pg0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751117AbeB0JI3 (ORCPT ); Tue, 27 Feb 2018 04:08:29 -0500 Received: by mail-pg0-f49.google.com with SMTP id g12so7290658pgs.0 for ; Tue, 27 Feb 2018 01:08:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:in-reply-to:references :organization:mime-version:content-transfer-encoding; bh=AqnGsX4ng5eju/LmOKHUeXvJu+ci/40GOFAT+8+6pFg=; b=N4GSBnnW6OIhYhLw+jj0QItZER87cM82PmMpgtUhRgmH+RPcEVkTU7b39ozzq3iA25 jUeoryGEjlJ7DZ5kFbPhn8LP3tcwK7o9Cn3OsHDMtVwr0HkfTwwnPBfRAZpvDB4HEgJD 3dzrIn2/seeShiUZnmGzEyyXCOqN+bYnsVN/wiSw1C/+HRJaGaLoCFh/9L7zwOOM6fxm Ooyyl1che04Wbh42ocHaOLEF1C2IHHj6+V7BkB7QrHStAWqsVkL9ggKkl09Bay+c/vwP 8VS1yLPlL8bV2rATVy+jQs98JbcJuPhEnFcmK7Vx2JU4ns7vMN8C9gjhlJiyzLhi3wX0 76SA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:organization:mime-version:content-transfer-encoding; bh=AqnGsX4ng5eju/LmOKHUeXvJu+ci/40GOFAT+8+6pFg=; b=E+X1rXaTFcSh/F2K2MyOh0LJCZxdtgWbWlpUhRXfJk0jWsCOiXfUsREvhz1DRq/pae iPpcYdONfgTIL7dH6Us01FxnEP+bkCoXqcTiKMnQGcWb+tjGwNWMQkrRP3s16jTA5qHQ pacVvRDFmMvWJUtdC4bk+lY+SEcLZTj2u0PeyMvwvhqFyXrKt//DG1n84OcOHrIEIvUE f78YC3ANzsST8s5wvJDBaTdEXQqIJkqQlSQyp3cdLAOSFZE+QifK+dgsvogXOV/Qhov3 ggO3Pg61HsNuG9zZPWAU6pjXBQPufQIyN5cSoGf/ZVudF8OAEKnxDZwHwGCNbQbvKbQJ NTZA== X-Gm-Message-State: APf1xPAXDMANBsZdlfBRjlzffbJ+wA5G2u83tT9YzrljP97UeDnxIVLC dNZTJDxbHf/EvuYKw3IBnv0= X-Received: by 10.98.67.78 with SMTP id q75mr13225770pfa.98.1519722508934; Tue, 27 Feb 2018 01:08:28 -0800 (PST) Received: from roar.ozlabs.ibm.com (115-64-218-172.tpgi.com.au. [115.64.218.172]) by smtp.gmail.com with ESMTPSA id q87sm12620317pfa.29.2018.02.27.01.08.25 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 27 Feb 2018 01:08:28 -0800 (PST) Date: Tue, 27 Feb 2018 19:08:18 +1000 From: Nicholas Piggin To: "Aneesh Kumar K.V" Cc: Christophe Leroy , linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Subject: Re: [RFC REBASED 4/5] powerpc/mm/slice: Use const pointers to cached slice masks where possible Message-ID: <20180227190818.3f3673e4@roar.ozlabs.ibm.com> In-Reply-To: <87h8q27uvi.fsf@linux.vnet.ibm.com> References: <02a62db83282b5ef3e0e8281fdc46fa91beffc86.1518382747.git.christophe.leroy@c-s.fr> <6a8c9183257dfcfeb1d8ed1ecc778ec3da19dcd4.1518382747.git.christophe.leroy@c-s.fr> <87h8q27uvi.fsf@linux.vnet.ibm.com> Organization: IBM X-Mailer: Claws Mail 3.16.0 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 27 Feb 2018 12:59:53 +0530 "Aneesh Kumar K.V" wrote: > Christophe Leroy writes: > > > The slice_mask cache was a basic conversion which copied the slice > > mask into caller's structures, because that's how the original code > > worked. In most cases the pointer can be used directly instead, saving > > a copy and an on-stack structure. > > > > This also converts the slice_mask bit operation helpers to be the usual > > 3-operand kind, which is clearer to work with. And we remove some > > unnecessary intermediate bitmaps, reducing stack and copy overhead > > further. > > Can we move the reduce unncessary intermediate bitmaps as another patch? This was rightly split out -- it moved to Christophe's patch he already made in his series. The changelog can just be adjusted. Thanks, Nick