On Tue, Feb 02, 2021 at 11:23:17AM +0000, Avri Altman wrote: > > > > > > On Tue, Feb 02, 2021 at 10:30:01AM +0200, Avri Altman wrote: > > > @@ -175,6 +179,8 @@ struct ufshpb_lu { > > > > > > /* for selecting victim */ > > > struct victim_select_info lru_info; > > > + struct work_struct ufshpb_normalization_work; > > > + unsigned long work_data_bits; > > > > You only have 1 "bit" being used here, so perhaps just a u8? Please > > don't use things like "unsigned long" for types, this isn't Windows :) > I am using it for atomic bit operations. Ah, and that requires "unsigned long"? Ok, nevermind :)