From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934428AbeEWV1U (ORCPT ); Wed, 23 May 2018 17:27:20 -0400 Received: from mail-io0-f195.google.com ([209.85.223.195]:34986 "EHLO mail-io0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933195AbeEWV1P (ORCPT ); Wed, 23 May 2018 17:27:15 -0400 X-Google-Smtp-Source: AB8JxZrmDsNYj42DBIrR/gkX+6/ZWGdRtlsVTruNpMNT3IA2t4PTQrN3mNpXUPtNJ9lAyKA+n+RNgA== Subject: Re: [PATCH] block drivers/block: Use octal not symbolic permissions To: Joe Perches , "Ed L. Cashin" , Philipp Reisner , Lars Ellenberg , Jiri Kosina , Josef Bacik , linux-block@vger.kernel.org, Ilya Dryomov , Sage Weil , Alex Elder , Joshua Morris , Philip Kelleher , "Michael S. Tsirkin" , Jason Wang , Konrad Rzeszutek Wilk , =?UTF-8?Q?Roger_Pau_Monn=c3=a9?= , Boris Ostrovsky , Juergen Gross Cc: linux-kernel@vger.kernel.org, drbd-dev@lists.linbit.com, nbd@other.debian.org, ceph-devel@vger.kernel.org, virtualization@lists.linux-foundation.org, xen-devel@lists.xenproject.org References: <5e97a292c1ba38a6b5a0caa271d85dc3de1b2aa7.1527105857.git.joe@perches.com> From: Jens Axboe Message-ID: <6bf44255-145f-bf14-e254-860731ff9296@kernel.dk> Date: Wed, 23 May 2018 15:27:11 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.0 MIME-Version: 1.0 In-Reply-To: <5e97a292c1ba38a6b5a0caa271d85dc3de1b2aa7.1527105857.git.joe@perches.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 5/23/18 2:05 PM, Joe Perches wrote: > Convert the S_ symbolic permissions to their octal equivalents as > using octal and not symbolic permissions is preferred by many as more > readable. > > see: https://lkml.org/lkml/2016/8/2/1945 > > Done with automated conversion via: > $ ./scripts/checkpatch.pl -f --types=SYMBOLIC_PERMS --fix-inplace > > Miscellanea: > > o Wrapped modified multi-line calls to a single line where appropriate > o Realign modified multi-line calls to open parenthesis Honestly, I see this as pretty needless churn. -- Jens Axboe