User Details
User Details
- User Since
- Nov 26 2016, 6:29 PM (395 w, 3 d)
Apr 6 2017
Apr 6 2017
Mikhail.Kupchik_gmail.com updated the diff for D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
Fix for a typo in comment: "read read" -> "read"
Mikhail.Kupchik_gmail.com updated the diff for D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
I just did add an small nit, see https://www.illumos.org/rb/r/403/diff/2-3/ just to be sure we wont get partial reads (which may happen with IO errors).
Mikhail.Kupchik_gmail.com added a comment to D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
What is the current status of this work?
Mar 19 2017
Mar 19 2017
Mikhail.Kupchik_gmail.com updated the diff for D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
fixed these two style issues
Mar 17 2017
Mar 17 2017
Mikhail.Kupchik_gmail.com updated the diff for D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
Added overflow prevention to fatget() function, submitted whole file diff.
Mikhail.Kupchik_gmail.com added a comment to D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
I've read this comment:
https://www.illumos.org/rb/r/403/#comment1770
Mikhail.Kupchik_gmail.com updated the diff for D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
Freeing of DOS_FS structure was moved from dos_mount() to dos_open(), where allocation happens. Also error handling in dos_open() was improved, resource leaks fixed.
Mar 16 2017
Mar 16 2017
Mikhail.Kupchik_gmail.com updated the diff for D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
indentation fixed at line 165
Mikhail.Kupchik_gmail.com updated the diff for D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
cstyle issues were fixed.
Feb 11 2017
Feb 11 2017
Mikhail.Kupchik_gmail.com retitled D9547: libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory from to libstand/dosfs: cache FAT32 in 128 Kb blocks to save loader memory.
Nov 26 2016
Nov 26 2016
Mikhail.Kupchik_gmail.com added a comment to D8644: loader: dosfs can corrupt memory.
I have tested this patch in my environment and confirm that it works as expected (fixes PR214423).