On Tue, Feb 02, 2021 at 11:23:17AM +0000, Avri Altman wrote:

quoted

On Tue, Feb 02, 2021 at 10:30:01AM +0200, Avri Altman wrote:

quoted

@@ -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 :)