On Tue, 2012-05-15 at 13:47 +0200, Joel Reardon wrote:
> For vol_id, its straightforward everywhere except: ubi_wl_init_scan()
> 
> where scheduling erase on the:
> 
> struct ubi_scan_leb *seb
> 
> appears to lack access to the volume id.  Should we add vol_id to
> ubi_scan_leb, and in ubi_scan() run:
> 
>         ubi_rb_for_each_entry(rb1, sv, &si->volumes, rb) {
>                 ubi_rb_for_each_entry(rb2, seb, &sv->root, u.rb)
> 			seb->vol_id = sv->vol_id;
> 
> Or is there a simpler way?

I guess yes. Add this field as a separate patch please.

-- 
Best Regards,
Artem Bityutskiy