From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f50.google.com (mail-wm1-f50.google.com [209.85.128.50]) (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 73AB22F8BD9 for ; Fri, 14 Nov 2025 09:06:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763111220; cv=none; b=ev6JH8jKcNP7f0dY2YzXnldDdB+MQIsy2lufQlXGhP1iArDZh43lYhIUs5x4CPZxNWVj1MSi9RBxL7LMBRqC/DwopabaPHVuIWy3y+6Nliku3XCgdvddvyCwn67Ebz4ZVQh0V8JOQQtpVrkHUgVybQ0Ohdi+6eTafcfwSm++NiU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763111220; c=relaxed/simple; bh=6zHnHDGEsf3utNSz4L3WgxRyx4n+9ITDWclyuCK6DGk=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=hJK13wh5A3GCZpY9npzSM1m0L5x7s3Qq7Wh3nvGTpyfgu0CtC5unYIhn6yOhgwETC954CVMii7QsMb2VAe5slkdWKY1fQ/HeVfaW8dL/12ISB8LuEMePaoXXcLZ70Rs9liLBJA6lC/FIt/JvjUt09IHQl6loOl6cDunAqiLW1qY= 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=EZPRT4Nk; arc=none smtp.client-ip=209.85.128.50 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="EZPRT4Nk" Received: by mail-wm1-f50.google.com with SMTP id 5b1f17b1804b1-4775ae5684fso8946725e9.1 for ; Fri, 14 Nov 2025 01:06:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1763111216; x=1763716016; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=MhzSvQe2CH4l6IIRGts1CDJxv3IbNpAVJaCMsNzVhko=; b=EZPRT4Nk+R1ZtfgfH3YHGNiO/q3I5lhXIQ1W3Cdl12r+W662W7fzX1XeoDd0AHDrlu rNQ7jvIR+a61u3B06zh416liOmzJ8G13VW04cMZMZOT8DqQ1yPCsl03P7AzRAHaUuCAR t8Nm5oOBNZI5oFXRrPvUhVnmwpxJZi8t2561ThaRcZvbtFsjmXcqCUm9uNPCah5gQVgu Zk4W/x51dI4iAVW5gTuVUJ6Zh7p3n8/K83L/6VF2GO3HEoV/6NZVT6cKP3QQ2xjWYLYR MgG2n4pnF6cvLmC1Xw31uDK+Hv4iS+fws+eCsUulVY1bJANQd32h2mtOxLNTppHz7GUK ObsA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763111216; x=1763716016; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=MhzSvQe2CH4l6IIRGts1CDJxv3IbNpAVJaCMsNzVhko=; b=o5BpzGVju6MhR+0bLCAifWaRbRbYYNhE9GVQPvGAfGtTZnNnUjOjp4ZuXiNzcGSJZM 6F4OR+3pUm5I7VJUd0AuORDCEi1QCQAgfVYJvEdfU7jm+o6v/FRjK5Uui/1lZFkSW7vv aHUMo8AfSqcrq8M88hGBqIQdJDl6izwr6lY3l+naqk0dyK1V5R6QlNPUD4u+MkvSs8eb GgKEPic/ooVSkx5OrY1EGR92tzkzO1ir6Sd420IkcmOY77/xx6xqMZIYpwpEz9AWVR3+ zJajVXjbdbJEp9wotoi4GyaMC7/2XgeMQN1tYA0bDDsmIh9735KhjyNcs5AaRlLPFPiH TbXw== X-Forwarded-Encrypted: i=1; AJvYcCX4Zs0o1suAaQYbMg9clRKe6aHFf9zRHbjJHuZqySKPfmZyS4RwgXfAb0nwpoZM/6JJcyd7jbdeGnGOgYA=@vger.kernel.org X-Gm-Message-State: AOJu0YxWVM6kPbsQo3PXrln2uVZo+V1rQKt5Nnpo2FahmkoI8h/l0cyz S+bXaM/uA7t8+m4esx00LNNIFMoqETW/u6FBDEKP28oI2Kw25cqa4GhQ X-Gm-Gg: ASbGncu8NSN5gVSazSyFLyLM/8YPhG8g/JA4Rm8gYbx7LmI0P3PTa88MD1E/L/RF4bv g3DgSA8mJaO/88kEujNpAnlzZEFZwXzlA7Xkg0DG4+p7i79YU2IDRDY+26xhbN9e6vcD5nGOXLY AKt6ah8UiW+sP1Qo6vNLr57MqqEUBevt9yv2c3prioqWc1aMZerDWZMOSh80SW7YeYjvAzeA3hJ z/I1iQHRz8VHx6BfT2xZmxH8YAgWC5B79K3vqQsl1SWIOQUnKg3N7zDrWjthpzwRip7B5NtHB/V gNDzH1Vi+IWSgJWnzSdOPOjpOkpwalg/VCFCeimuf5iYuulZAYxezl0EcnUzjC7ldGOjGSqQ5rn ZsALywboTO/fzSIpbXXthwmG3ydHFh/JANVSgIpqPGTjxRkE2KYwN7aIVvSHOHzr0aKoSMEyEin Ndd48VBR7g X-Google-Smtp-Source: AGHT+IFI+ClLTh8e5PZwbJHazQxmDRFj0ZSGRbSblC0Z/GkqM10glV9hJcbfKBsbw2e3ySHwMKi/uQ== X-Received: by 2002:a05:600c:3b19:b0:477:8a2a:1244 with SMTP id 5b1f17b1804b1-4778fe5c5f9mr19412595e9.11.1763111215765; Fri, 14 Nov 2025 01:06:55 -0800 (PST) Received: from [192.168.1.187] ([161.230.67.253]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-42b53e7b074sm8484141f8f.7.2025.11.14.01.06.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 14 Nov 2025 01:06:55 -0800 (PST) Message-ID: <5cd213e377dbf6ff0fb24f92cefd0e4283375d17.camel@gmail.com> Subject: Re: [PATCH] iio: buffer: use dma_buf_unmap_attachment_unlocked() helper From: Nuno =?ISO-8859-1?Q?S=E1?= To: Liang Jie , Jonathan Cameron , David Lechner , Nuno =?ISO-8859-1?Q?S=E1?= , Andy Shevchenko , "open list:IIO SUBSYSTEM AND DRIVERS" , open list Cc: liangjie@lixiang.com, fanggeng@lixiang.com, yangchen11@lixiang.com Date: Fri, 14 Nov 2025 09:07:33 +0000 In-Reply-To: <20251114084725.2079196-1-buaajxlj@163.com> References: <20251114084725.2079196-1-buaajxlj@163.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.58.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Fri, 2025-11-14 at 16:47 +0800, Liang Jie wrote: > From: Liang Jie >=20 > Replace open-coded dma_resv_lock()/dma_resv_unlock() around > dma_buf_unmap_attachment() in iio_buffer_dmabuf_release() with the > dma_buf_unmap_attachment_unlocked() helper. >=20 > This aligns with the standard DMA-BUF API, avoids duplicating > locking logic and eases future maintenance. No functional change. >=20 > Reviewed-by: fanggeng > Signed-off-by: Liang Jie > --- Reviewed-by: Nuno S=C3=A1 > =C2=A0drivers/iio/industrialio-buffer.c | 4 +--- > =C2=A01 file changed, 1 insertion(+), 3 deletions(-) >=20 > diff --git a/drivers/iio/industrialio-buffer.c b/drivers/iio/industrialio= -buffer.c > index a80f7cc25a27..58330abbf40a 100644 > --- a/drivers/iio/industrialio-buffer.c > +++ b/drivers/iio/industrialio-buffer.c > @@ -1563,9 +1563,7 @@ static void iio_buffer_dmabuf_release(struct kref *= ref) > =C2=A0 struct iio_buffer *buffer =3D priv->buffer; > =C2=A0 struct dma_buf *dmabuf =3D attach->dmabuf; > =C2=A0 > - dma_resv_lock(dmabuf->resv, NULL); > - dma_buf_unmap_attachment(attach, priv->sgt, priv->dir); > - dma_resv_unlock(dmabuf->resv); > + dma_buf_unmap_attachment_unlocked(attach, priv->sgt, priv->dir); > =C2=A0 > =C2=A0 buffer->access->detach_dmabuf(buffer, priv->block); > =C2=A0