On 12.01.23 10:06, Christoph Hellwig wrote: > Move the code that splits the ordered extents and records the physical > location for them to the storage layer so that the higher level consumers > don't have to care about physical block numbers at all. This will also > allow to eventually remove accounting for the zone append write sizes in > the upper layer with a little bit more block layer work. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> > Reviewed-by: Naohiro Aota <naohiro.aota@xxxxxxx> > Reviewed-by: Josef Bacik <josef@xxxxxxxxxxxxxx> > Reviewed-by: Johannes Thumshirn <johannes.thumshirn@xxxxxxx> What base is this against? I'm getting the following: Applying: btrfs: handle recording of zoned writes in the storage layer fs/btrfs/inode.c: In function ‘btrfs_submit_direct’: fs/btrfs/inode.c:7842:25: error: label ‘out_err’ used but not defined 7842 | goto out_err; | ^~~~ Specifically that hunk needs to be removed: @@ -7911,7 +7888,6 @@ static void btrfs_submit_direct(const struct iomap_iter *iter, out_err_em: free_extent_map(em); -out_err: dio_bio->bi_status = status; btrfs_dio_private_put(dip); } But trivial to fix up when applying.