Commit eacb2574 authored by Kent Overstreet's avatar Kent Overstreet Committed by Kent Overstreet

bcachefs: bch_dev->dev

Add a field to bch_dev for the dev_t of the underlying block device -
this fixes a null ptr deref in tracepoints.
Signed-off-by: default avatarKent Overstreet <kent.overstreet@gmail.com>
parent d8601afc
...@@ -432,6 +432,7 @@ struct bch_dev { ...@@ -432,6 +432,7 @@ struct bch_dev {
struct bch_sb_handle disk_sb; struct bch_sb_handle disk_sb;
struct bch_sb *sb_read_scratch; struct bch_sb *sb_read_scratch;
int sb_write_error; int sb_write_error;
dev_t dev;
struct bch_devs_mask self; struct bch_devs_mask self;
......
...@@ -568,8 +568,11 @@ static long bch2_ioctl_disk_get_idx(struct bch_fs *c, ...@@ -568,8 +568,11 @@ static long bch2_ioctl_disk_get_idx(struct bch_fs *c,
if (!capable(CAP_SYS_ADMIN)) if (!capable(CAP_SYS_ADMIN))
return -EPERM; return -EPERM;
if (!dev)
return -EINVAL;
for_each_online_member(ca, c, i) for_each_online_member(ca, c, i)
if (ca->disk_sb.bdev->bd_dev == dev) { if (ca->dev == dev) {
percpu_ref_put(&ca->io_ref); percpu_ref_put(&ca->io_ref);
return i; return i;
} }
......
...@@ -1257,6 +1257,8 @@ static int __bch2_dev_attach_bdev(struct bch_dev *ca, struct bch_sb_handle *sb) ...@@ -1257,6 +1257,8 @@ static int __bch2_dev_attach_bdev(struct bch_dev *ca, struct bch_sb_handle *sb)
ca->disk_sb = *sb; ca->disk_sb = *sb;
memset(sb, 0, sizeof(*sb)); memset(sb, 0, sizeof(*sb));
ca->dev = ca->disk_sb.bdev->bd_dev;
percpu_ref_reinit(&ca->io_ref); percpu_ref_reinit(&ca->io_ref);
return 0; return 0;
...@@ -1875,7 +1877,7 @@ struct bch_dev *bch2_dev_lookup(struct bch_fs *c, const char *path) ...@@ -1875,7 +1877,7 @@ struct bch_dev *bch2_dev_lookup(struct bch_fs *c, const char *path)
rcu_read_lock(); rcu_read_lock();
for_each_member_device_rcu(ca, c, i, NULL) for_each_member_device_rcu(ca, c, i, NULL)
if (ca->disk_sb.bdev->bd_dev == dev) if (ca->dev == dev)
goto found; goto found;
ca = ERR_PTR(-ENOENT); ca = ERR_PTR(-ENOENT);
found: found:
......
...@@ -387,7 +387,7 @@ TRACE_EVENT(alloc_scan, ...@@ -387,7 +387,7 @@ TRACE_EVENT(alloc_scan,
), ),
TP_fast_assign( TP_fast_assign(
__entry->dev = ca->disk_sb.bdev->bd_dev; __entry->dev = ca->dev;
__entry->found = found; __entry->found = found;
__entry->inc_gen = inc_gen; __entry->inc_gen = inc_gen;
__entry->inc_gen_skipped = inc_gen_skipped; __entry->inc_gen_skipped = inc_gen_skipped;
...@@ -409,7 +409,7 @@ TRACE_EVENT(invalidate, ...@@ -409,7 +409,7 @@ TRACE_EVENT(invalidate,
), ),
TP_fast_assign( TP_fast_assign(
__entry->dev = ca->disk_sb.bdev->bd_dev; __entry->dev = ca->dev;
__entry->offset = offset, __entry->offset = offset,
__entry->sectors = sectors; __entry->sectors = sectors;
), ),
...@@ -431,7 +431,7 @@ DECLARE_EVENT_CLASS(bucket_alloc, ...@@ -431,7 +431,7 @@ DECLARE_EVENT_CLASS(bucket_alloc,
), ),
TP_fast_assign( TP_fast_assign(
__entry->dev = ca->disk_sb.bdev->bd_dev; __entry->dev = ca->dev;
__entry->reserve = reserve; __entry->reserve = reserve;
), ),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment