[PATCH 13/25] mtd: Convert to dynamically allocated bdi infrastructure

2017-04-12 Thread Jan Kara
MTD already allocates backing_dev_info dynamically. Convert it to use generic infrastructure for this including proper refcounting. We drop mtd->backing_dev_info as its only use was to pass mtd_bdi pointer from one file into another and if we wanted to keep that in a clean way, we'd have to make

Re: [PATCH 13/25] mtd: Convert to dynamically allocated bdi infrastructure

2017-04-12 Thread Jan Kara
On Wed 12-04-17 01:14:06, Christoph Hellwig wrote: > > + sb->s_bdi = bdi_get(mtd_bdi); > > + sb->s_iflags |= SB_I_DYNBDI; > > FTI, while I think this is a faithful conversion of the existing code, > the single bdi for all MTD devices looks rather bogus to me.. Yeah, I don't understand why

Re: [PATCH 13/25] mtd: Convert to dynamically allocated bdi infrastructure

2017-04-12 Thread Christoph Hellwig
> + sb->s_bdi = bdi_get(mtd_bdi); > + sb->s_iflags |= SB_I_DYNBDI; FTI, while I think this is a faithful conversion of the existing code, the single bdi for all MTD devices looks rather bogus to me.. Otherwise this looks good: Reviewed-by: Christoph Hellwig

[PATCH 13/25] mtd: Convert to dynamically allocated bdi infrastructure

2017-03-29 Thread Jan Kara
MTD already allocates backing_dev_info dynamically. Convert it to use generic infrastructure for this including proper refcounting. We drop mtd->backing_dev_info as its only use was to pass mtd_bdi pointer from one file into another and if we wanted to keep that in a clean way, we'd have to make