Page MenuHomeFreeBSD

Mikhail.Kupchik_gmail.com (Mikhail Kupchik)
User

Projects

User does not belong to any projects.

User Details

User Since
Nov 26 2016, 6:29 PM (395 w, 3 d)

Recent Activity

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"

Apr 6 2017, 10:32 AM
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).

Apr 6 2017, 10:24 AM
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?

Apr 6 2017, 9:39 AM

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 19 2017, 10:24 PM

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.

Mar 17 2017, 8:04 PM
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

Mar 17 2017, 7:37 PM
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 17 2017, 4:23 PM

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

Mar 16 2017, 10:39 PM
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.

Mar 16 2017, 9:59 PM

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.
Feb 11 2017, 6:56 PM

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).

Nov 26 2016, 6:31 PM