Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 30 Jul 2021 11:06:39 -0300
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Xiyu Yang <xiyuyang19@fudan.edu.cn>
Cc:     Mike Marciniszyn <mike.marciniszyn@cornelisnetworks.com>,
        Dennis Dalessandro <dennis.dalessandro@cornelisnetworks.com>,
        Doug Ledford <dledford@redhat.com>, linux-rdma@vger.kernel.org,
        linux-kernel@vger.kernel.org, yuanxzhang@fudan.edu.cn,
        Xin Tan <tanxin.ctf@gmail.com>
Subject: Re: [PATCH] RDMA/hfi1: Convert from atomic_t to refcount_t on
 hfi1_devdata->user_refcount
Message-ID: <20210730140639.GB2559559@nvidia.com>
References: <1626674454-56075-1-git-send-email-xiyuyang19@fudan.edu.cn>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1626674454-56075-1-git-send-email-xiyuyang19@fudan.edu.cn>
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from mlx.ziepe.ca (206.223.160.26) by YT1PR01CA0050.CANPRD01.PROD.OUTLOOK.COM (2603:10b6:b01:2e::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4373.21 via Frontend Transport; Fri, 30 Jul 2021 14:06:40 +0000
Received: from jgg by mlx with local (Exim 4.94)        (envelope-from <jgg@nvidia.com>)        id 1m9T9r-00AjsI-1o; Fri, 30 Jul 2021 11:06:39 -0300
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: a8ddb9f3-2ed6-498b-47a4-08d9536340b4
X-MS-TrafficTypeDiagnostic: BL1PR12MB5362:
X-Microsoft-Antispam-PRVS: <BL1PR12MB53629F590380D1AC526E46A3C2EC9@BL1PR12MB5362.namprd12.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:4502;
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: eFB6mH494DcEGVPV+GM23Oja1FWaCPgQ2R6CxG6XQPWj0cos/vLzcMmqDl2jh3cZ+gv9AJ9sThX+XheS8fCfEDOcVIZJGTkLQPfI/1sNvBbRx6xNA2hhw0nKDmZcYOCWr0ALncuUuOYM4nV35dB4fg62lIJl7Lp89F6vk/b+doUlayEQ8ZP57o0N4/TlYXi9a+rBQzO/YAGJSzVGh6FYZgoxFqRsVMkRliWoinuobS4WdJq2PTt2IsNG7K5Inykz7Q5fPi16K/DLbwZGX2hak7nzeb4Jzw1rQYRf1g7agxnMHEb6EQzColeqRbFI3sgwLynjpXauzBctGvfyMjDNdxKn92Wrcmc/C4EBrjcMdcogMCUcL8NI8Z+w3Ez00o9cxhMzqpYCRy92YgQ1g3tFTrG6z5VBvuHw71I8q0iKFlFEjEXedOaOVfMtEF4bmNbdoDw1sk3CwO1PugCpXVDBPTbl1n0Nvy1lWWzxsplL6zjDYp7ALJssTleNIr5MWZ0D2y7bRqpMv8J6IOXPotuKoeXXZ0C0xo3sdzMxl6RQ62RELpl7bgrMRegObPU0M7Cg0btCFyMM0s8OMrVC0BeerTTO2fAfSU6ShHBAx55PVd8zF58EyKndDiVQy1GFrVqYoJ5gysRj2VzuREmN8Lb3JQ==
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR12MB5506.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(136003)(346002)(39860400002)(376002)(396003)(366004)(9786002)(9746002)(33656002)(8936002)(38100700002)(54906003)(6916009)(426003)(186003)(66946007)(83380400001)(2616005)(66476007)(66556008)(478600001)(4744005)(1076003)(8676002)(316002)(36756003)(4326008)(86362001)(26005)(2906002)(5660300002);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?XOeocBEUfnkTqniQgoqbveBVaCvpgm7yxw0tSUSAJdvjGEFY3yZPvkfGWXC8?=
 =?us-ascii?Q?M7WbuCZn3SbZk/NpfiQWTlRtxwhdjaT1AXYlP16wKLtL2ktGc1PiAyCYVOKs?=
 =?us-ascii?Q?PRN7F0FQQraacjCPXMqk4clloTZ4dyqUg2eXavpRxP2BSO4FA6LKcGHxooQM?=
 =?us-ascii?Q?sqqUZ/MsJ88Gws7ljEMhm6hhWEAyfusoBesvIBHYSr1mRxdAujpfhpwEOZXl?=
 =?us-ascii?Q?d7UzSfyx5qCr4U0cNFhGMakwI6SBv4IibUCuB9ttgrTPhLHwQKQeHuo8OOnZ?=
 =?us-ascii?Q?E4o6/VvcVYqPSs6SgOJb/BGthZtzSMdV+6tunCHMAQyiiHevcrDNuzFxRM/x?=
 =?us-ascii?Q?WcOuF1cA89JmHJojkNRfviRtXKUPhv9J+sPFV36N9obylgVNbVi2A23xjvt2?=
 =?us-ascii?Q?D7WWXGY8XRCYrEbOFkc1osGkiPMrCxFvbEZ62rBhI+dlBmbUXg3xG+k/P43c?=
 =?us-ascii?Q?sTQtZIYCOiTwmdsZWHVC8EiWzfRj+s1nEMvBj9NUlv+hFD/ifB9RZkQ/bkKD?=
 =?us-ascii?Q?XzFebigB4Vwb5ZH8v9SNUXz8okqLfbJF6/Zv+Rj4N07itnJBCdByCEw6WMWD?=
 =?us-ascii?Q?VbXm9r8AF93FxCIvWLGx/XTqzu47eZV/KDUkvh3pgo+mCCZEg/c6j6/XXCzV?=
 =?us-ascii?Q?eYZk7Y+wiB9ohXN77V2jC9G1bybKg4XYuqZegb74+/77qE8rL6SIUTDS2q20?=
 =?us-ascii?Q?lGwXqs3pAwid9OUGGQMhtW2TbchR8rVlsIIiB9eZmVea4l/0p0TUI39+2Vd6?=
 =?us-ascii?Q?Rqqx7/8cNopljCpnrKlp0C2n6ZcElWxqq6RUOCExcjjeHy9+vTLH4yLV2jEu?=
 =?us-ascii?Q?IWQzAzozknmiZTzNJ+CSEhN3kGIXUFK8VhE2LSc605yAt6J291mjQrhgm/9d?=
 =?us-ascii?Q?1ZGZzm1w/sfPe7dOfFtCqiDJxfuqYlq0qlSPfqkwXTbuW3D78RKWqTcLHOLk?=
 =?us-ascii?Q?TP3JNAFUhcei28ZbZEn62XdxYvUFsavnjBJC4QduZ91goN2FkXRi2r6ikDZ4?=
 =?us-ascii?Q?AOEuhDYE4UGeOOBHdnp/VpObvMBByCKkzMwQyXK1eKWvXC0E+HCjZWGZcEuR?=
 =?us-ascii?Q?CTN05DTC+zzTJF4Uvgh3UbfxfurRbsFiEREG4c9zZ5KNkBKC958Nq6yBCW63?=
 =?us-ascii?Q?8F4ktZ0+xvj7cYHgcJoz0FjStqolZzDNo8j9pBxeFtBSuKnqqwtXHhnxBSad?=
 =?us-ascii?Q?rHtXepTcH0qcfRDGDtn242dv3UuokrpRuCcker7vSqGqart0PDEK2bpQb76W?=
 =?us-ascii?Q?8OqC0jlnIbtBtyeAQPUd8bF71Dt/ZFbPFCVWQp2yi6+B0+j8pjMsnuefID6g?=
 =?us-ascii?Q?PbHkQ/9wz9AEgOY5iyt7n/SA?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a8ddb9f3-2ed6-498b-47a4-08d9536340b4
X-MS-Exchange-CrossTenant-AuthSource: BL0PR12MB5506.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Jul 2021 14:06:40.4439
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: tkn8l2790a4xRe4H6ez+bF5hGk1AEGnFMiZqFzGGCbyx0CgNCE9Vl5bhKQc7UfpJ
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5362
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Jul 19, 2021 at 02:00:53PM +0800, Xiyu Yang wrote:
> refcount_t type and corresponding API can protect refcounters from
> accidental underflow and overflow and further use-after-free situations.
> 
> Signed-off-by: Xiyu Yang <xiyuyang19@fudan.edu.cn>
> Signed-off-by: Xin Tan <tanxin.ctf@gmail.com>
> Tested-by: Josh Fisher <josh.fisher@cornelisnetworks.com>
> Acked-by: Dennis Dalessandro <dennis.dalessandro@cornelisnetworks.com>
> ---
>  drivers/infiniband/hw/hfi1/chip.c     | 2 +-
>  drivers/infiniband/hw/hfi1/file_ops.c | 6 +++---
>  drivers/infiniband/hw/hfi1/hfi.h      | 3 ++-
>  drivers/infiniband/hw/hfi1/init.c     | 2 +-
>  4 files changed, 7 insertions(+), 6 deletions(-)

Applied to for-next, thanks

Jason