From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-il1-f175.google.com (mail-il1-f175.google.com [209.85.166.175]) (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 2CB2E28935A for ; Wed, 7 May 2025 14:01:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.166.175 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746626518; cv=none; b=eBBcJ/yA6GmTyCzM3ydFoH0ZZG3PLCTqXZE9zIq/sE2wshl3vM0DVFlAmHyztamFuPqHxubLKQ6BLNYxUuZG0Jk678hIf/Rw7BicRo/fXqM1/0E6+z5pNF5wu1fCfM0FRmrxhXPXmyz55aXCbmwmeQCW3bg9a+jfJpMshuiOAr4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746626518; c=relaxed/simple; bh=riUohbe+nhSKSfWHET/Yt5mBIMTKqf2kkbHWdODQalU=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=Ns+2Fa77F6W0X0ISPtujcUlNyX2RRbqARNZ7D1KENwhK70uNyMhNC8VMesNVGEHZerIsmn15w6JiaBt63DcoFJ2Kyxcgv/VKzaeO+OdJFMmBbflVEhpV3jiS5c7zpvCRsBctsyxxzQxbMN/MD4ScG3hq64wIy3TU/4pGLUWlmbE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.dk; spf=pass smtp.mailfrom=kernel.dk; dkim=pass (2048-bit key) header.d=kernel-dk.20230601.gappssmtp.com header.i=@kernel-dk.20230601.gappssmtp.com header.b=F5xPJ/Ip; arc=none smtp.client-ip=209.85.166.175 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.dk Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=kernel.dk Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel-dk.20230601.gappssmtp.com header.i=@kernel-dk.20230601.gappssmtp.com header.b="F5xPJ/Ip" Received: by mail-il1-f175.google.com with SMTP id e9e14a558f8ab-3d96d16b375so36674595ab.1 for ; Wed, 07 May 2025 07:01:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel-dk.20230601.gappssmtp.com; s=20230601; t=1746626515; x=1747231315; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:content-language:from :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=md71yFj7QjYOD6CmfKKjAp0TbUSgcGH/TtN+850EMGU=; b=F5xPJ/IpbgIrYmCyxL1ZnXNjxIAq/NJXLETWQ0tRAZhF9QkonvCKWNcxAgQQjfwfgZ xMm6Vf/rJcGELq5T0W9kzb8p55lxGhW5frDG45H9pQjUtv7CN3CvxjUcf3HFoZ/cadzU vJOxFDXiqvKtr8Nz1k5NJjJClGpbkwpHZYgruWMyUUuZRKBhANyKM0fgYS1FaiSZG1XT RQibs2LGpUvxPSPGxVNb6uqdEdE2SUzGvYvDgjLWF60NhTShvumr+ATlvvlCQa/bd7V4 OUOAkW27boFix23FyJTulJxbi9TkikYZtgr17wtlL5l1pMwFpKjRpgsKiKiU9WLcbWuq 2DSw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1746626515; x=1747231315; h=content-transfer-encoding:in-reply-to:content-language:from :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=md71yFj7QjYOD6CmfKKjAp0TbUSgcGH/TtN+850EMGU=; b=OFwS+CqgTdrrXx4ISK6mdFDvxIaZ7Z/QDl90rl/GmC2aqpkBpb41eAVb7gHRzoVKPn KVdIVJzNHNfS/ll3G3IkeniAr4qGmQ5Xbd4BTiujcq/vfPJDrWvgualR5F9gikYxI2bq PWjzf7ifd3iGr2rwQpgEvygcjf5Z/bfGii27zCMVjv45RKtMhmd1RBHmrE6DDI6KAW1b SUEDGWf6oe5zAF1qxsgiZHKQSHHlyBLA4oPA/kjyV4EV7+dQiRw7lstweV/3Skxe67vl 9RXCjV2Yx6WonxNSGy9Wh6XarKC904nAshB+BrftVk0BvgvfVqieyRrt3LAClhTXJNNm rXBA== X-Forwarded-Encrypted: i=1; AJvYcCXtIumfJ2mg0fLTachXtjs9TQTC6NnhNk46BSr5fj4dyZOcuia8OeYTJnVs9a6TWW05x5SJ85O4yA==@vger.kernel.org X-Gm-Message-State: AOJu0YyqrY6hvFQluWG+bgiHVn+bJ/XjJaoWnOJ277HkX7dgAKqmd/qL 8dwKBzmN83pOLu1qIVnUABg4Gtdfvc9+VgRuoGtnc2r2RuZNp9J+Hdyv9fnPesM= X-Gm-Gg: ASbGnctt8lJvCnvFne37+mhOVPI/zfT+NxTdCCwfN5XWomwQM+yZ5oGg9Jv7fP3IdJt XSka+lYYNm60gNetvkkQI37L2TVJ8pYGSU8+v0m5ohNwyPQPpf4TGSRelXAFmCx4orH/dUQ9579 pYFW5wR1HJell6ETw5af1gR1JRL0hKOpXcoElk/ocorVNYqH+CH5OYTRThq67Dc+NjbXHoW5pmy pkJ63khLpTDGwS095L0xhs0Y59g5IhzfveHnee3tTWmdaMwNprhU8P01w8pJz6sazgjwcI4VID8 fbs+paKvuJuJzLhsU0pR8bFY06t5fpm7GHgS X-Google-Smtp-Source: AGHT+IEPXkcUdjuoTWJKGESqgsxo1A4ob6I1ttYfFC6JrLgpCILRh6jZ2+LBOAWlTXtq2pbhrFyjbA== X-Received: by 2002:a05:6e02:1529:b0:3d4:36c3:7fe3 with SMTP id e9e14a558f8ab-3da738e562emr32326165ab.9.1746626515075; Wed, 07 May 2025 07:01:55 -0700 (PDT) Received: from [192.168.1.116] ([96.43.243.2]) by smtp.gmail.com with ESMTPSA id e9e14a558f8ab-3d975f6dd89sm31362225ab.66.2025.05.07.07.01.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 07 May 2025 07:01:54 -0700 (PDT) Message-ID: Date: Wed, 7 May 2025 08:01:52 -0600 Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 02/19] block: add a bdev_rw_virt helper To: Christoph Hellwig Cc: linux-block@vger.kernel.org, "Md. Haris Iqbal" , Jack Wang , Coly Li , Kent Overstreet , Mike Snitzer , Mikulas Patocka , Chris Mason , Josef Bacik , David Sterba , Andreas Gruenbacher , Carlos Maiolino , Damien Le Moal , Naohiro Aota , Johannes Thumshirn , "Rafael J. Wysocki" , Pavel Machek , slava@dubeyko.com, glaubitz@physik.fu-berlin.de, frank.li@vivo.com, linux-bcache@vger.kernel.org, dm-devel@lists.linux.dev, linux-btrfs@vger.kernel.org, gfs2@lists.linux.dev, linux-fsdevel@vger.kernel.org, linux-xfs@vger.kernel.org, linux-pm@vger.kernel.org, Hannes Reinecke , Johannes Thumshirn References: <20250507120451.4000627-1-hch@lst.de> <20250507120451.4000627-3-hch@lst.de> From: Jens Axboe Content-Language: en-US In-Reply-To: <20250507120451.4000627-3-hch@lst.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 5/7/25 6:04 AM, Christoph Hellwig wrote: > +/** > + * bdev_rw_virt - synchronously read into / write from kernel mapping > + * @bdev: block device to access > + * @sector: sector to access > + * @data: data to read/write > + * @len: length in byte to read/write > + * @op: operation (e.g. REQ_OP_READ/REQ_OP_WRITE) > + * > + * Performs synchronous I/O to @bdev for @data/@len. @data must be in > + * the kernel direct mapping and not a vmalloc address. > + */ > +int bdev_rw_virt(struct block_device *bdev, sector_t sector, void *data, > + size_t len, enum req_op op) I applied the series, but did notice a lot of these - I know some parts like to use the 2-tab approach, but I still very much like to line these up. Just a style note for future patches, let's please have it remain consistent and not drift towards that. -- Jens Axboe