On 30.12.24 12:28, Pavel Tikhomirov wrote:


On 12/6/24 05:55, Alexander Atanasov wrote:
@@ -794,8 +794,8 @@ static void ploop_advance_local_after_bat_wb(struct ploop *ploop,
          }
          if (success) {
-            bat_entries[i] = dst_clu[i];
-            md->bat_levels[i] = ploop_top_level(ploop);
+            bat_entries[i] = READ_ONCE(dst_clu[i]);

Should not it be WRITE_ONCE above?

Yes, it is fixed in dm-ploop: fix access to batpages to use WRITE_ONCE.
Patches are marked for merging.


+            WRITE_ONCE(md->bat_levels[i], ploop_top_level(ploop));
          } else {
              ploop_hole_set_bit(i + off, ploop);
          }


--
Regards,
Alexander Atanasov

_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to