From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753236AbdHWCVT (ORCPT ); Tue, 22 Aug 2017 22:21:19 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:39203 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752965AbdHWCVS (ORCPT ); Tue, 22 Aug 2017 22:21:18 -0400 To: Todd Poynor Cc: Doug Gilbert , "James E.J. Bottomley" , "Martin K. Petersen" , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Hannes Reinecke Subject: Re: [PATCH] sg: protect against races between mmap() and SG_SET_RESERVED_SIZE From: "Martin K. Petersen" Organization: Oracle Corporation References: <20170816054108.119890-1-toddpoynor@google.com> Date: Tue, 22 Aug 2017 22:21:06 -0400 In-Reply-To: <20170816054108.119890-1-toddpoynor@google.com> (Todd Poynor's message of "Tue, 15 Aug 2017 22:41:08 -0700") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Source-IP: userv0022.oracle.com [156.151.31.74] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Todd, > Take f_mutex around mmap() processing to protect against races with > the SG_SET_RESERVED_SIZE ioctl. Ensure the reserve buffer length > remains consistent during the mapping operation, and set the > "mmap called" flag to prevent further changes to the reserved buffer > size as an atomic operation with the mapping. Applied to 4.14/scsi-queue (with a slight whitespace fix). Thanks! -- Martin K. Petersen Oracle Linux Engineering