Project

General

Profile

Bug #70102

Merge fix for act_scan_laundry_weight mechanism

Added by Alexander Motin almost 2 years ago. Updated almost 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Alexander Motin
Category:
OS
Target version:
Seen in:
Severity:
Low Medium
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

I've found that 11.1-stable branch misses one-line fix for memory deactivation logic, that may affect distribution between active, inactive and laundry memory.


Related issues

Related to FreeNAS - Bug #35515: Laundry ram/memory queue is not being cleaned out by the pagedaemonClosed

History

#1 Updated by Alexander Motin almost 2 years ago

  • Status changed from In Progress to Ready for Testing
  • Target version changed from Backlog to TrueNAS 11.1-U8
  • Needs Merging changed from Yes to No

PR: https://github.com/freenas/os/pull/172

QE: I am not sure how to better test this, but it would be good to run some stress-tests with user-space memory pressure, like Samba, VMs, Jails, etc, or all together, looking for some unusual memory management artifacts, like excessive swapping that was not happening before, etc.

#2 Updated by Alexander Motin almost 2 years ago

  • Related to Bug #35515: Laundry ram/memory queue is not being cleaned out by the pagedaemon added

#3 Updated by Dru Lavigne almost 2 years ago

  • Target version changed from TrueNAS 11.1-U8 to 11.1-U7

#4 Updated by Dru Lavigne almost 2 years ago

  • Needs Doc changed from Yes to No

#7 Updated by Caleb St. John almost 2 years ago

Based on conversation with Bonne, this was tested on customer system running close to 10k unique smbd sessions. The box is stable and the laundry mechanism still works as intended after applying this patch. I would consider this "tested".

#8 Updated by Bonnie Follweiler almost 2 years ago

  • Status changed from Ready for Testing to Passed Testing
  • Needs QA changed from Yes to No

#9 Updated by Dru Lavigne almost 2 years ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF