Received: by 2002:ac0:aed5:0:0:0:0:0 with SMTP id t21csp6997022imb; Sat, 9 Mar 2019 12:28:19 -0800 (PST) X-Google-Smtp-Source: APXvYqzH29lNRaqNJSlmm188biYNmb49OoNe6RvRgqPfkeA/uAP/6cEACzdUWWqXbDd0L11yovSd X-Received: by 2002:a17:902:784c:: with SMTP id e12mr25503460pln.117.1552163299250; Sat, 09 Mar 2019 12:28:19 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1552163299; cv=none; d=google.com; s=arc-20160816; b=YpPhXFtjNCB+O8q/GKm8Z7vGQuPQW7XtCWbOLwaIcksQ5op6lWPkcVzQ4wfslKb99J wdNtxVk1uBXY2xek42Qmi0ZLmZaRYKdBUyBkrIzI9/N+cE7et6uv3y+IxxLcuStBB0xx K62/E4o4wFssaisYHf8cydi36z3mriQAAWp12UbQVKtZaba86VQoayoSsft1eUFaj3Rv byAlKYoT3yksN5j+JQ/9lkYi7adzKSzGv+eNWtE7t3ZJ7IwbjLRMMRfl3F0EvXeWO8fz JXHev2irDoLAaGphbkzp5AItm8oPSBz1M1L3HO6BruJ8HWgqjX6FQ7nXH8rpPAZDsYwH 58Xw== 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:message-id:date :mime-version:subject:references:in-reply-to:cc:to:from :dkim-signature; bh=8FG/9gLREUJAOj5pH8rp0s0swEnerCijUGE6Ohzjl/M=; b=zrKddIDZAXrM6G26XwqM6Af9+DAg199gWILCF1+zSSu7sYFEPyhAAjuQ7szPgjhwkg AedxsgdsEedugVOx00mLb2t0AIABYdYzm0gAqrINQ+DU+JdlRp0LWBw/XAq35t/paowM viuy+rf67qVpbKplVFoV4YEawjPTXETtD+9mrYln2w7aE4SMBNy7aHiPPaKqWQsz3jZg +EkHIJ08oP4wlR9ai4UuopHJgSq7vpzBiXWkU9RAPGs6/2sD5SGfFyF2kbteZSOGcBl3 qLPkdi3WUQHm5RPZ7zJfYfLWZ5SQLUboBcLk4uPsJyqG/XtCOrnyKRJzlutQ/hZCig8I Snvw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@yandex.ru header.s=mail header.b=Z8o81IH2; 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=NONE dis=NONE) header.from=yandex.ru Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 91si1300335pla.14.2019.03.09.12.27.50; Sat, 09 Mar 2019 12:28:19 -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=@yandex.ru header.s=mail header.b=Z8o81IH2; 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=NONE dis=NONE) header.from=yandex.ru Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726402AbfCIU1S (ORCPT + 99 others); Sat, 9 Mar 2019 15:27:18 -0500 Received: from forward500o.mail.yandex.net ([37.140.190.195]:57288 "EHLO forward500o.mail.yandex.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726340AbfCIU1S (ORCPT ); Sat, 9 Mar 2019 15:27:18 -0500 X-Greylist: delayed 440 seconds by postgrey-1.27 at vger.kernel.org; Sat, 09 Mar 2019 15:27:16 EST Received: from mxback15o.mail.yandex.net (mxback15o.mail.yandex.net [IPv6:2a02:6b8:0:1a2d::66]) by forward500o.mail.yandex.net (Yandex) with ESMTP id 713F1600BD; Sat, 9 Mar 2019 23:19:52 +0300 (MSK) Received: from localhost (localhost [::1]) by mxback15o.mail.yandex.net (nwsmtp/Yandex) with ESMTP id gUC8cd86v7-JneedDhV; Sat, 09 Mar 2019 23:19:50 +0300 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yandex.ru; s=mail; t=1552162790; bh=8FG/9gLREUJAOj5pH8rp0s0swEnerCijUGE6Ohzjl/M=; h=Message-Id:Cc:Subject:In-Reply-To:Date:References:To:From; b=Z8o81IH2y1SXL8ipRnhiKYErNurmH1xqaiE7fHw7Waa4oJAOUK6pspl9fzrRZju6d qjbSAYC/tZVI5mJ5Q48+CxCDPwHWhJt59xln2qaygMOJ2HjCGFw7QubuaN4IQdPaAY yAFg2Vp3ORmWIc1xKUZ2CGgx0PhZ9U3EsgC6nP/s= Authentication-Results: mxback15o.mail.yandex.net; dkim=pass header.i=@yandex.ru Received: by myt6-2fee75662a4f.qloud-c.yandex.net with HTTP; Sat, 09 Mar 2019 23:19:49 +0300 From: Andrey Abramov To: "lkml@sdf.org" , "andriy.shevchenko@linux.intel.com" Cc: "akpm@linux-foundation.org" , "daniel.wagner@siemens.com" , "dchinner@redhat.com" , "don.mullis@gmail.com" , "geert@linux-m68k.org" , "linux-kernel@vger.kernel.org" , "linux@rasmusvillemoes.dk" In-Reply-To: <201903091553.x29FrfMR018600@sdf.org> References: , , <20190309140653.GO9224@smile.fi.intel.com> <201903091553.x29FrfMR018600@sdf.org> Subject: Re: [PATCH 1/5] lib/sort: Make swap functions more generic MIME-Version: 1.0 X-Mailer: Yamail [ http://yandex.ru ] 5.0 Date: Sat, 09 Mar 2019 23:19:49 +0300 Message-Id: <10806701552162789@myt6-2fee75662a4f.qloud-c.yandex.net> Content-Transfer-Encoding: 7bit Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > Although I'm thinking of: > > static bool __attribute_const__ > is_aligned(const void *base, size_t size, unsigned char align) > { > unsigned char lsbits = (unsigned char)size; > > (void)base; > #ifndef CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS > lsbits |= (unsigned char)(uintptr_t)base; > #endif > return (lsbits & (align - 1)) == 0; > } > > Any preference? I think it would be better. > I find "u32s" confusing; I keep reading the "s" as "signed" rather than a plural. How about one of: swap_bytes / swap_ints / swap_longs swap_1 / swap_4 / swap_8 In my opinion "swap_bytes / swap_ints / swap_longs" are the most readable. (Good job)