Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 15 Jul 2022 12:28:47 +0300
From:   Dan Carpenter <dan.carpenter@oracle.com>
To:     binyi <dantengknight@gmail.com>
Cc:     Joe Perches <joe@perches.com>, Manish Chopra <manishc@marvell.com>,
        GR-Linux-NIC-Dev@marvell.com, Coiby Xu <coiby.xu@gmail.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        netdev@vger.kernel.org, linux-staging@lists.linux.dev,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] staging: qlge: Fix indentation issue under long for
 loop
Message-ID: <20220715092847.GU2316@kadam>
References: <20220710210418.GA148412@cloud-MacBookPro>
 <20220712134610.GO2338@kadam>
 <1d6fd2b271dfa0514ccb914c032e362bc4f669fa.camel@perches.com>
 <20220715060457.GA2928@cloud-MacBookPro>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220715060457.GA2928@cloud-MacBookPro>
User-Agent: Mutt/1.9.4 (2018-02-28)
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?SjZkZ39o0zjxqWc6G9Q3Q6VfKIMWCVrpUS8VBTDTjuMoX6GPYX3dsMhZCl/g?=
 =?us-ascii?Q?yO7YN0/ca9zgXKTVAakxnowWNAfg0VtXesV7EHNKQmxH7esafxkWBnXj//ut?=
 =?us-ascii?Q?mRa/K1zpTafow7enJ+AhrWuDCdp5e2TviJ0QpddKX/hkmOmDNDjNw+ynBDHc?=
 =?us-ascii?Q?R0k2qVPJgEAvtnKqH1bpSozFNXVisxQIqoFwyZbvoNIok9tGxoDLKQlWiCyF?=
 =?us-ascii?Q?KqSN9RNtvCY9sw9kRB55HvFImDNijiIETuAkbVsxWQFG145Y/Zsxq20eDDNg?=
 =?us-ascii?Q?AxZsM6bRRW1hWM+m83ES1PIsP8nkNWpSX9LYr72X7H6yLScF2d5MnupecPer?=
 =?us-ascii?Q?URpwJXSTLt2hfxpqO/gy4rsti82AAUhFVPUK/4hWJaTLSDxvKvC37wl0kLVr?=
 =?us-ascii?Q?qss7N01Jx30p3XELb7D2G2TeDUIfafCJoj6BCMrFT+JGk5HOPP+c2NYU836E?=
 =?us-ascii?Q?2HhPGNfhExAdBTsle2JFk9ul7RfBcuk6wngsjlBnegzermZi8hEjzaU9S3a4?=
 =?us-ascii?Q?3u2EPQ5M5YRVwVIxC4F4CQ3XHwRSjhA0+6gd/puf0oJT//zZuAmCePSCc2vm?=
 =?us-ascii?Q?oc6yFPUNR0n2tw8sMajSP9J9O1NGM363yjRkEaSJvfLqmDHwDEQG+cvc1oWS?=
 =?us-ascii?Q?X0IdTgxnr2wEs4bqpEovIlwHBvrHDmy4L/Zx5S6A/0dKo3rYCsIuZ+xCvzkR?=
 =?us-ascii?Q?pYro3O/doiFhAUi6+FLeekseH83FoIXUCtId4JQiwk+t4ACCbge4x//WZwva?=
 =?us-ascii?Q?YVIEFgHtKZhC4mKnrumcSGlbpwSUVUxJZ5e9IJziON+eQUfdfIcSQBIQWp/T?=
 =?us-ascii?Q?YtohaJZeL24agP6lUdfxpysiCvSnFdNHmNIjma5K6BbqfNccueEdyH4NT7yV?=
 =?us-ascii?Q?Aml1Yils7V8YJS9jzkru+v7hwzmHJmgInUYD0TRD/giDKyKbnq6WO1drQYEI?=
 =?us-ascii?Q?VAqoV0a2Yc9RdhUWW/k725PTNTwZZqIx0cYtcZ7wh2khma+coSR5LI2+uGt6?=
 =?us-ascii?Q?dbOJz8qX2LVmCQxZ/VELMuvNSyTBa1JGkKDsciWJHImYtryK8cigft3rROXl?=
 =?us-ascii?Q?cursahMQsCPEiyvSC85/m/iQ7MmgZVhMp07WgSMVzxqiVrnQpChT27VHQB0S?=
 =?us-ascii?Q?8HacKx2yXRop4/4DUMoAAkaH+z/7YKdiShRxW4EUxHWRzBcX5E0650GBU20s?=
 =?us-ascii?Q?n3YXHTwK51cNs8JpVcUHn06F6ZDdGQ20zTpYU2YqeZDTnaVt40sSXhjW3jUP?=
 =?us-ascii?Q?DHblvuptqllsFh1TmGalKsoiSQ4CUPF6X3ehaYoGpxiIgb6S2T1cmzshbgP/?=
 =?us-ascii?Q?z+VKxLyVF3RJYNQepAe4QIKJeqlknMRfOvc0V8b9uPt+z39pLbJFjnhDfevM?=
 =?us-ascii?Q?dFmHt9RPqPCNFIZ6kMXQefmqhfG8pN1jye8/AoG2spIXOyzcSEuqdJcs4kY4?=
 =?us-ascii?Q?tZrRX0Be4E+R95hIXMl5gCL6/K/qix6d3K7O94nytshhSO75rYq59B3BZL9D?=
 =?us-ascii?Q?3suGwrploedyBp2ZrcZeJFZAlaLRK81dvKAS8psa1tVC/yzmwSXRjraaBo9H?=
 =?us-ascii?Q?yKa33Qaxk5VBav9tCmjrB7IFjIqbRcmGJ6Qk/uUbUpGcBXz+mn++m7VKF1VC?=
 =?us-ascii?Q?JQ=3D=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9001cd0b-6e40-4fec-8a45-08da66447151
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jul 2022 09:28:57.5074
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: losszNnB+osTZfwz72IUV2neQsZnUIp+DoXJkFrIXbtWhCzvHYTENbX3t+SGXvikkuc/C999OPESC8uV9tdFgKnr4z5qWSgM5jCaK3UOYmM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR10MB1318
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.517,18.0.883
 definitions=2022-07-15_03:2022-07-14,2022-07-15 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 phishscore=0
 mlxlogscore=895 suspectscore=0 adultscore=0 mlxscore=0 spamscore=0
 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2206140000 definitions=main-2207150040
X-Proofpoint-ORIG-GUID: GTzj75gADcHa4YHr4QgrSK2XCy0x9NBF
X-Proofpoint-GUID: GTzj75gADcHa4YHr4QgrSK2XCy0x9NBF
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Jul 14, 2022 at 11:04:57PM -0700, binyi wrote:
> On Tue, Jul 12, 2022 at 07:14:55AM -0700, Joe Perches wrote:
> > On Tue, 2022-07-12 at 16:46 +0300, Dan Carpenter wrote:
> > > > @@ -3007,10 +3007,12 @@ static int qlge_start_rx_ring(struct qlge_adapter *qdev, struct rx_ring *rx_ring
> > > >  		tmp = (u64)rx_ring->lbq.base_dma;
> > > >  		base_indirect_ptr = rx_ring->lbq.base_indirect;
> > > >  
> > > > -		for (page_entries = 0; page_entries <
> > > > -			MAX_DB_PAGES_PER_BQ(QLGE_BQ_LEN); page_entries++)
> > > > -				base_indirect_ptr[page_entries] =
> > > > -					cpu_to_le64(tmp + (page_entries * DB_PAGE_SIZE));
> > > > +		for (page_entries = 0;
> > > > +		     page_entries < MAX_DB_PAGES_PER_BQ(QLGE_BQ_LEN);
> > > > +		     page_entries++) {
> > > > +			base_indirect_ptr[page_entries] = cpu_to_le64(tmp);
> > > > +			tmp += DB_PAGE_SIZE;
> > > 
> > > I've previously said that using "int i;" is clearer here.  You would
> > > kind of expect "page_entries" to be the number of entries, so it's kind
> > > of misleading.  In other words, it's not just harmless wordiness and
> > > needless exposition, it's actively bad.
> > 
> > Likely true.
> > 
> 
> I agree it could be misleading. But if "page_entries" is in the for loop I
> would assume it's some kind of index variable, and still it provides some
> information (page entry) for the index, probably page_entry_idx could be
> better name but still makes the for loop a very long one. I guess I would
> leave it be.

It does not "provide some information".  That's what I was trying to
explain.  It's the opposite of information.  Information is good. No
information is neutral.  But anti-information is bad.

Like there are so many times in life where you listen to someone and you
think you are learning something but you end up stupider than before.
They have done studies on TV news where it can make you less informed
than people who don't watch it.  And other studies say if you stop
watching TV news for even a month then your brain starts to heal.

I don't really care about one line of code, but what I'm trying to say
is learn to recognize anti-information and delete it.  Comments for
example are often useless.

regards,
dan carpenter