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