From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f53.google.com (mail-wm1-f53.google.com [209.85.128.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8897A3AA9F8 for ; Tue, 2 Jun 2026 11:21:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780399271; cv=none; b=QD+nCW0VNv2KOl0ZVFSUDSFcJ2gUnX4ZC1gpAndBjII8ACCB+VO2CubDkESCeiZBNnTIakI2XTzinh61O21Uipnmvsdh3SoxY1dB1/kAgf8HLwO9mzB8n02oTnAF8nzCglrGksl5LHklEoPVh53I43Y97uNtc7+r95IeqYSU26A= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780399271; c=relaxed/simple; bh=mdyShz2Hb1nhbf16JMmhzAI3lw9uTjlKGNRSkU4NhIY=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=RZfQ2UUV2IRh2VvIEjVdQ3/904Z47y0INRv7948fXK4r1r560OrL5VTI3L9aoR95H74NgBC4bqQ8OqF0Uz5x12CbUs/g/8krp3dhBLgnOrF7/LFvVykCXKewvoWUenp5jT5Xy+65LS3xs5s6Gb6Al4j6diAOr6gwdUUExzbgNPg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=QWDREzUc; arc=none smtp.client-ip=209.85.128.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="QWDREzUc" Received: by mail-wm1-f53.google.com with SMTP id 5b1f17b1804b1-490ace40f4bso20843505e9.3 for ; Tue, 02 Jun 2026 04:21:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1780399269; x=1781004069; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=N+Mx9qxe5Noh2BseN2qEXUEhNI+HpIudHixhjkzy9lQ=; b=QWDREzUcNvwFfuiuy45CUK0AXWlgdkktBJ+GtFJ8FBXCqyxwgJFlKRhsPar2d2ZSS/ wF+OEbhaLwx7LV28ZPNWspdgAJm9xUvN96CZwuoAtPTffSCzW/4Fpr6/Mbt9enzRoXPK A8vQoiFTbt5B2mNhEoq1uAz7t7WnR63xJyAs9Jqz6Qq8pOBKts+L5/hKYYv7x9LFkOQk hLy4CUB02VWElNjR5IpQWxMuofdoGXhNgBfSxRh30zfh3WNMYPoKoGEmTte5sN9ugr58 IEZwZrfmGvoplpLC/y3c2bT2VXy/rXJ4VUqMPgGRnElRSiMysIq4uug9vXoy1EJesNkh bggA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1780399269; x=1781004069; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=N+Mx9qxe5Noh2BseN2qEXUEhNI+HpIudHixhjkzy9lQ=; b=BXeMdZoZGrqY6mEs38ppDd6wkH208qYxRY/eyWpOEszoRQ/cVubwwDZGcG11J7xzhx b58jovl8NgR4f0yYPt6+zIXxe+qLJ148tmybmzEzI7Y0oqIuy4drUb9r2tG4nm9iTrZL +CmCvIqbLLWz2dZz1wj8i15Uo2vkjxZqxBiteSvelDBZKxElZDBP1J7mpqemf//Tei3N /SP3i7DBHBvUixRDoUdSMkoFhs8PFpcjRXPKo3yHe/KR3/hX82W3akOTstioaLB81xST +ZZHgxRPzAFYS4dS+fNIQcQWjFgVLq6/fkZk7QDH5qQ/oi/E37lXOK1ox155EbGlwYBp f43w== X-Forwarded-Encrypted: i=1; AFNElJ8XzNL0c5H9GkfCVhak/43fZbp0qeV3z+PQ1DZq5EZ2HUxbh8Y+nk6m2JZo+cBRR0MB8RakYnwHcaFH9xI=@vger.kernel.org X-Gm-Message-State: AOJu0YyvkUYAukxYdtzU4jTPFrQjT1QeQIxbOkDSsQJ+cZisR95UXOoB Ng+uUkfCXobi+KjUZO/aodGH9vhFYDhCJ9R68HEojaDA5oUceB+QWYg0 X-Gm-Gg: Acq92OEZNIeQ+Amm4R9xGKpYgAxI/P8FjBEE0xjmBX9BVncvetkf4riT4ErdujRcGj0 bB+ngnLczw+kwBbxKdFx8Qw+rAgHX9/vUA7gyMeA5zl212rlJOnNr5y2JtwOcgnuxHNmBkrC5iy 0ggMkEZ8x7d8Qd6uJN5nstEauKKQ/x/NtNT+5B3LT6AQtWLHzEUjHmRZm/L5NWbGgBKrbyw8hpZ ZTZMCtFZpit8ZZ+yUXmajo1b9Cx8E5t1oksEd/6Ax+nZwfcbvwO3sD9idJ5z35C+I9n5yHlfDne Cb0Gd8hJgicTsEMXa3wc0js9qaAZavjHNyigTNtr5+jP0RHXPsQOJKgLzqSEHPtCX5lqD8Enezl SxeOicBGtgub3cEpd2iJBU8HjMfRdDeoXqTLwz7JPnBz9rxxU9rEEzH+/iFmAHDQFx9TMPNplR4 EWZcgKDyBxsFJqGnx29YYGD0IiV4matrLZ8D7TPT5pdAjwYXsDscCGkx+OWmCNGwJaWG7itS4= X-Received: by 2002:a05:600c:46d1:b0:490:a7ab:bbe3 with SMTP id 5b1f17b1804b1-490a7abbd7fmr206981415e9.0.1780399268568; Tue, 02 Jun 2026 04:21:08 -0700 (PDT) Received: from pumpkin (82-69-66-36.dsl.in-addr.zen.co.uk. [82.69.66.36]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-45ef3559645sm31033836f8f.26.2026.06.02.04.21.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 02 Jun 2026 04:21:08 -0700 (PDT) Date: Tue, 2 Jun 2026 12:21:04 +0100 From: David Laight To: Andy Shevchenko Cc: Bart Van Assche , Erni Sri Satya Vennela , mkalderon@marvell.com, Jason Gunthorpe , Leon Romanovsky , zyjzyj2000@gmail.com, sagi@grimberg.me, mgurtovoy@nvidia.com, haris.iqbal@ionos.com, jinpu.wang@ionos.com, kbusch@kernel.org, Jens Axboe , Christoph Hellwig , kch@nvidia.com, smfrench@gmail.com, linkinjeon@kernel.org, metze@samba.org, tom@talpey.com, chuck.lever@oracle.com, jlayton@kernel.org, neil@brown.name, okorniev@redhat.com, Dai.Ngo@oracle.com, trondmy@kernel.org, anna@kernel.org, achender@kernel.org, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, horms@kernel.org, kees@kernel.org, ebadger@purestorage.com, linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org, target-devel@vger.kernel.org, linux-nvme@lists.infradead.org, linux-cifs@vger.kernel.org, samba-technical@lists.samba.org, linux-nfs@vger.kernel.org, netdev@vger.kernel.org, rds-devel@oss.oracle.com, Jason Gunthorpe Subject: Re: [PATCH rdma-next v6] RDMA: Change capability fields in ib_device_attr from int to u32 Message-ID: <20260602122104.20afa8b4@pumpkin> In-Reply-To: References: <20260601092534.1764560-1-ernis@linux.microsoft.com> <5d3cac2b-4011-49c5-a142-55c85d38e90f@acm.org> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; arm-unknown-linux-gnueabihf) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Tue, 2 Jun 2026 12:21:58 +0300 Andy Shevchenko wrote: > On Mon, Jun 01, 2026 at 08:51:40AM -0700, Bart Van Assche wrote: > > On 6/1/26 2:25 AM, Erni Sri Satya Vennela wrote: > > ... > > > > - sdev->srq_size = min(srpt_srq_size, sdev->device->attrs.max_srq_wr); > > > + sdev->srq_size = min_t(u32, srpt_srq_size, sdev->device->attrs.max_srq_wr); > > > > min_t() shouldn't be used if there is an alternative available. For the > > SRP drivers, please make sure that both arguments of min() are unsigned > > instead of using min_t(). > > Ah, I just answered in similar way against v5. I also mentioned clamp() there. > IMHO it is also best to do min(value, 255) not min(255, value). Like an 'if' put the value you are comparing against second. The min_t(u8, x, y) you've removed are usually broken. Maybe I should change clamp() to allow clamp(int_var, 0, unsigned_var). That will need the order of the compares swapping (to do the low bound first). I think they used to be that way around, got changed by a commit that said it didn't change it! Correct code shouldn't care. -- David