mav (Alexander Motin)
User

Projects

User Details

User Since
Jun 3 2014, 6:27 PM (207 w, 4 d)

Recent Activity

Yesterday

mav added a comment to D15562: ZFS sorted scans.

I don't think it is a very good idea to merge two pretty much independent ZoL commits into one FreeBSD commit. It could have sense if the first one would be significantly broken and second fixed the issue, but priority upgrade implemented in second commit is only a performance optimization, mostly unrelated to scrub even, in which case having it separate would make commit history cleaner for later comparison.

Sat, May 26, 3:03 AM · ZFS

Fri, May 25

mav accepted D15552: add support for console resuming, implement it for uart, use on x86.

Makes sense to me.

Fri, May 25, 3:30 PM
mav committed rS334200: Refactor NVMe CAM integration..
Refactor NVMe CAM integration.
Fri, May 25, 3:34 AM

Thu, May 24

mav added a comment to D15517: We can't release the refcount outside of the periph lock..

As I have told, I have no objections. I still don't like the global counters, since they mean we can never remove respective lock or at least atomics, but for now I can live with that. It is better then broken code.

Thu, May 24, 2:01 PM

Wed, May 23

mav committed rS334099: Add ready polling after PHY reset on VIA SATA controllers..
Add ready polling after PHY reset on VIA SATA controllers.
Wed, May 23, 3:23 PM

Tue, May 22

mav accepted D15517: We can't release the refcount outside of the periph lock..

Looks good to me. The only rough edge I see is that outstanding_cmds is used only for the newly added assertion, that is why it was so broken before this and nobody care.

Tue, May 22, 6:11 PM

Mon, May 21

mav added a comment to D15446: AES CCM-CBC cryptography code.

Great to see it! I did only first on-surface look so far, so comments are mostly cosmetic.

Mon, May 21, 4:39 PM

Thu, May 17

mav accepted D15456: change ffs_blkfree() trim to not bypass geom_vfs, avoids panics.

Good catch.

Thu, May 17, 12:33 AM
mav accepted D15453: Make reference counting more robust.

It is OK to me. I still don't think protection against GEOM bugs belongs here, but so be it, since we any way have and acquire lock to protect that.

Thu, May 17, 12:31 AM

Wed, May 16

mav added a comment to D15413: stop and restart kernel event timers in the suspend / resume cycle.

I agree with John that this is better then nothing, but the proper way I think would be: first suspend third-party devices, then stop clock, and then suspend timer devices, resume -- in opposite order.

Wed, May 16, 12:11 AM

Mon, May 14

mav accepted D15422: calibrate lapic timer in native_lapic_setup.

The motivation for postponing the calibration was to not spend whole second on boot in cases when LAPIC timer is not used. Though that should be much less frequent lately.

Mon, May 14, 3:33 PM

Wed, May 9

mav committed rS333415: MFC r333158:.
MFC r333158:
Wed, May 9, 3:22 PM

Mon, May 7

mav committed rS333322: Keep CARP state as INIT when net.inet.carp.allow=0..
Keep CARP state as INIT when net.inet.carp.allow=0.
Mon, May 7, 2:45 PM
mav closed D14477: Keep CARP state as INIT when net.inet.carp.allow=0.
Mon, May 7, 2:45 PM
mav added a comment to D15305: hpet_suspend: reset configuration of all timers.

Something made me to comment out hpet_disable() call on suspend 8 years ago. Wish I remember what was that.

Mon, May 7, 1:45 PM

Sat, May 5

mav committed rS333275: Add NVMe Namespace Management support to nvmecontrol(8)..
Add NVMe Namespace Management support to nvmecontrol(8).
Sat, May 5, 8:08 PM
mav closed D11399: nvme: Add namespace mgmt and attachment support..
Sat, May 5, 8:08 PM

Fri, May 4

mav accepted D9299: Rework CTL frontend & backend options to use nv(3), allow creating multiple ioctl frontend ports.

Some more man/help tunings and so be it.

Fri, May 4, 6:19 PM
mav accepted D15276: Add virtio-scsi block storage backend support.

It looks good enough for me. I see some rough edges in details of SCSI interface implementation (for example, there seem to be a race window between regular and task management requests processing, submitted via different queues), but those may be addressed/improved later.

Fri, May 4, 3:48 PM
mav accepted D15291: for (generic) bus suspend, detach and shutdown iterate children in reverse order.

This looks like a hack, but I have no real objections.

Fri, May 4, 2:20 PM

Wed, May 2

mav committed rS333180: Fix LOR between controller and queue locks..
Fix LOR between controller and queue locks.
Wed, May 2, 8:13 PM
mav committed rS333158: Clean enclosure_table when resetting num_enc_table_entries to zero..
Clean enclosure_table when resetting num_enc_table_entries to zero.
Wed, May 2, 2:41 AM

Mon, Apr 30

mav committed rS333130: Improve nvme(4) attach/detach sequences..
Improve nvme(4) attach/detach sequences.
Mon, Apr 30, 11:06 PM
mav committed rS333127: Fix use-after-free in nvme_qpair_destroy()..
Fix use-after-free in nvme_qpair_destroy().
Mon, Apr 30, 9:28 PM
mav committed rS333123: Set si_drv1 for nvmeXnsY in a new race-free way..
Set si_drv1 for nvmeXnsY in a new race-free way.
Mon, Apr 30, 7:21 PM

Apr 25 2018

mav requested changes to D9299: Rework CTL frontend & backend options to use nv(3), allow creating multiple ioctl frontend ports.

Looks mostly fine aside of commented places.

Apr 25 2018, 12:40 AM

Apr 19 2018

mav committed rS332785: MFC r332523: 9433 Fix ARC hit rate.
MFC r332523: 9433 Fix ARC hit rate
Apr 19 2018, 5:56 PM
mav committed rS332779: Release memory resource on cuda driver attach failure..
Release memory resource on cuda driver attach failure.
Apr 19 2018, 3:29 PM

Apr 18 2018

mav committed rS332739: MFC r332641: MFV man pages update from r329502:.
MFC r332641: MFV man pages update from r329502:
Apr 18 2018, 8:32 PM

Apr 17 2018

mav committed rS332641: MFV man pages update from r329502: 7614 zfs device evacuation/removal..
MFV man pages update from r329502: 7614 zfs device evacuation/removal.
Apr 17 2018, 2:34 AM

Apr 16 2018

mav committed rS332554: MFC r331950: 9434 Speculative prefetch is blocked by device removal code..
MFC r331950: 9434 Speculative prefetch is blocked by device removal code.
Apr 16 2018, 4:17 AM
mav committed rS332553: MFC r331713: MFV r331712:.
MFC r331713: MFV r331712:
Apr 16 2018, 4:16 AM
mav committed rS332552: MFC r331711: MFV 331710:.
MFC r331711: MFV 331710:
Apr 16 2018, 4:15 AM
mav committed rS332551: MFC r331709: MFV r331708:.
MFC r331709: MFV r331708:
Apr 16 2018, 4:15 AM
mav committed rS332550: MFC r331707: MFV r331706:.
MFC r331707: MFV r331706:
Apr 16 2018, 4:14 AM
mav committed rS332549: MFC r331705: MFV 331704:.
MFC r331705: MFV 331704:
Apr 16 2018, 4:13 AM
mav committed rS332548: MFC r331703: MFV 331702:.
MFC r331703: MFV 331702:
Apr 16 2018, 4:12 AM
mav committed rS332547: MFC r331701: MFV r331695, 331700: 9166 zfs storage pool checkpoint.
MFC r331701: MFV r331695, 331700: 9166 zfs storage pool checkpoint
Apr 16 2018, 4:11 AM
mav committed rS332545: MFC r331699: Partial MFV r329753:.
MFC r331699: Partial MFV r329753:
Apr 16 2018, 3:58 AM
mav committed rS332544: MFC r331420 (by avg): zfs: fix mismatch between format specifier and type.
MFC r331420 (by avg): zfs: fix mismatch between format specifier and type
Apr 16 2018, 3:56 AM
mav committed rS332543: MFC r331414: Reduce struct aggsum_bucket padding to fit into one cache line..
MFC r331414: Reduce struct aggsum_bucket padding to fit into one cache line.
Apr 16 2018, 3:55 AM
mav committed rS332542: MFC r331408: MFV r331407: 9213 zfs: sytem typo.
MFC r331408: MFV r331407: 9213 zfs: sytem typo
Apr 16 2018, 3:54 AM
mav committed rS332541: MFC r331406: MFV r331405: 9084 spa_*_ashift must ignore spare devices.
MFC r331406: MFV r331405: 9084 spa_*_ashift must ignore spare devices
Apr 16 2018, 3:53 AM
mav committed rS332540: MFC r331404: MFV r331400:.
MFC r331404: MFV r331400:
Apr 16 2018, 3:53 AM
mav committed rS332539: MFC r329808: MFV r329807:.
MFC r329808: MFV r329807:
Apr 16 2018, 3:49 AM
mav committed rS332538: MFC r329805: MFV r329803:.
MFC r329805: MFV r329803:
Apr 16 2018, 3:48 AM
mav committed rS332537: MFC r329802: MFV r329799, r329800:.
MFC r329802: MFV r329799, r329800:
Apr 16 2018, 3:48 AM
mav committed rS332536: MFC r329798: MFV r329793, r329795:.
MFC r329798: MFV r329793, r329795:
Apr 16 2018, 3:45 AM
mav committed rS332535: MFC r329783: 8942 zfs promote .../%recv should be an error.
MFC r329783: 8942 zfs promote .../%recv should be an error
Apr 16 2018, 3:44 AM
mav committed rS332534: MFC r329777: MFV r329776:.
MFC r329777: MFV r329776:
Apr 16 2018, 3:43 AM
mav committed rS332533: MFC r329775: MFV r329774:.
MFC r329775: MFV r329774:
Apr 16 2018, 3:43 AM
mav committed rS332532: MFC r329771: MFV r329770: 9035 zfs: this statement may fall through.
MFC r329771: MFV r329770: 9035 zfs: this statement may fall through
Apr 16 2018, 3:42 AM
mav committed rS332531: MFC r329769: MFV r329766: 8962 zdb should work on non-idle pools.
MFC r329769: MFV r329766: 8962 zdb should work on non-idle pools
Apr 16 2018, 3:41 AM
mav committed rS332530: MFC r329765: MFV r329762: 8961 SPA load/import should tell us why it failed.
MFC r329765: MFV r329762: 8961 SPA load/import should tell us why it failed
Apr 16 2018, 3:40 AM
mav committed rS332529: MFC r329761: MFV r329760: 7638 Refactor spa_load_impl into several functions.
MFC r329761: MFV r329760: 7638 Refactor spa_load_impl into several functions
Apr 16 2018, 3:39 AM
mav committed rS332528: MFC r329759:.
MFC r329759:
Apr 16 2018, 3:39 AM
mav committed rS332526: MFC r329755: MFV r329753:.
MFC r329755: MFV r329753:
Apr 16 2018, 3:35 AM
mav committed rS332525: MFC r329732: MFV r329502: 7614 zfs device evacuation/removal.
MFC r329732: MFV r329502: 7614 zfs device evacuation/removal
Apr 16 2018, 3:33 AM
mav committed rS332524: MFC r307317: MFV r307313:.
MFC r307317: MFV r307313:
Apr 16 2018, 3:15 AM
mav committed rS332523: 9433 Fix ARC hit rate.
9433 Fix ARC hit rate
Apr 16 2018, 12:55 AM

Apr 13 2018

mav committed rS332461: MFC r332145: Do not fail devices just for errors in descriptor format..
MFC r332145: Do not fail devices just for errors in descriptor format.
Apr 13 2018, 12:30 AM

Apr 11 2018

mav added a comment to D14980: pathconf(_PC_MEDIA) - an API to tell the media at a given path.

Is this API and device classification exist somewhere else, or being just designed? Differentiation between STRIPE, MIRROR and RAID sound somewhat odd to me, same as between SPINNING and CDROM. Besides it duplicates "GEOM::rotation_rate" attribute, we already have and use in some places (though obviously it is not passed through UFS).

Apr 11 2018, 7:20 PM

Apr 9 2018

mav added a comment to D15010: add white listing for ZFS locking pairs that WITNESS can't report accurately and enable WITNESS by default in ZFS.

I am not closely familiar with WITNESS, so just a feeling: the long lists of blessed locks and their combinations promises high chances for them to be forgotten on following ZFS updates. At very list it would be good to document how those new mechanisms should be used.

Apr 9 2018, 5:07 PM

Apr 6 2018

mav committed rS332145: Do not fail devices just for errors in descriptor format..
Do not fail devices just for errors in descriptor format.
Apr 6 2018, 7:48 PM

Apr 4 2018

mav committed rS332041: Restore Sean Eric Fagan (sef) src commit bit..
Restore Sean Eric Fagan (sef) src commit bit.
Apr 4 2018, 6:27 PM

Apr 3 2018

mav committed rS331950: 9434 Speculative prefetch is blocked by device removal code..
9434 Speculative prefetch is blocked by device removal code.
Apr 3 2018, 9:16 PM
mav committed rS331903: MFC r331228: Update mpr(4) driver from v15 to v18 from Broadcom site..
MFC r331228: Update mpr(4) driver from v15 to v18 from Broadcom site.
Apr 3 2018, 2:29 AM

Apr 1 2018

mav committed rS331870: MFC r330292: Update QLogic ISP 24xx/25xx chips firmware to 8.07.00..
MFC r330292: Update QLogic ISP 24xx/25xx chips firmware to 8.07.00.
Apr 1 2018, 11:00 PM

Mar 31 2018

mav accepted D14913: Switch to NetBSD's implementation of El Torito EFI handling.
Mar 31 2018, 2:19 PM

Mar 29 2018

mav committed rS331721: MFC r329738: MFV r329736: 8969 Cannot boot from RAIDZ with parity > 1.
MFC r329738: MFV r329736: 8969 Cannot boot from RAIDZ with parity > 1
Mar 29 2018, 2:25 AM

Mar 28 2018

mav committed rS331713: MFV r331712:.
MFV r331712:
Mar 28 2018, 11:17 PM
mav committed rS331712: 9280 Assertion failure while running removal_with_ganging test with 4K devices.
9280 Assertion failure while running removal_with_ganging test with 4K devices
Mar 28 2018, 11:12 PM
mav committed rS331711: MFV 331710:.
MFV 331710:
Mar 28 2018, 11:06 PM
mav committed rS331710: 9188 increase size of dbuf cache to reduce indirect block decompression.
9188 increase size of dbuf cache to reduce indirect block decompression
Mar 28 2018, 10:57 PM
mav committed rS331709: MFV r331708:.
MFV r331708:
Mar 28 2018, 10:50 PM
mav committed rS331708: 9321 arc_loan_compressed_buf() can increment arc_loaned_bytes by the wrong value.
9321 arc_loan_compressed_buf() can increment arc_loaned_bytes by the wrong value
Mar 28 2018, 10:44 PM
mav committed rS331707: MFV r331706:.
MFV r331706:
Mar 28 2018, 10:29 PM
mav committed rS331706: 9235 rename zpool_rewind_policy_t to zpool_load_policy_t.
9235 rename zpool_rewind_policy_t to zpool_load_policy_t
Mar 28 2018, 10:17 PM
mav committed rS331705: MFV 331704:.
MFV 331704:
Mar 28 2018, 10:10 PM
mav committed rS331704: 9191 dump vdev tree to zfs_dbgmsg when spa load fails due to missing log devices.
9191 dump vdev tree to zfs_dbgmsg when spa load fails due to missing log devices
Mar 28 2018, 10:09 PM
mav committed rS331703: MFV 331702:.
MFV 331702:
Mar 28 2018, 10:07 PM
mav committed rS331702: 9187 racing condition between vdev label and spa_last_synced_txg in….
9187 racing condition between vdev label and spa_last_synced_txg in…
Mar 28 2018, 10:06 PM
mav committed rS331701: MFV r331695, 331700: 9166 zfs storage pool checkpoint.
MFV r331695, 331700: 9166 zfs storage pool checkpoint
Mar 28 2018, 10:01 PM
mav committed rS331700: Add files missed from r331695..
Add files missed from r331695.
Mar 28 2018, 9:00 PM
mav committed rS331699: Partial MFV r329753:.
Partial MFV r329753:
Mar 28 2018, 8:41 PM
mav committed rS331695: 9166 zfs storage pool checkpoint.
9166 zfs storage pool checkpoint
Mar 28 2018, 6:12 PM

Mar 26 2018

mav accepted D14828: Avoid a race when we're setting the event timer while going idle.

It slightly increases lock hold time, which may be a bottleneck in case of a global lock/timer, but it probably does not worth overcomplication to avoid only that.

Mar 26 2018, 3:55 PM
mav added a comment to D14827: Tighten up event scheduling.
In D14827#311953, @jtl wrote:

OK, if you think this will be a problem, I'll gladly accept suggestions that maintain correctness and improve performance.

Mar 26 2018, 3:28 PM

Mar 24 2018

mav added a comment to D14827: Tighten up event scheduling.

I'll need to look on it closer to recall all the details, but so far my main worry about your patches is that you are increase scope of ET_HW_LOCK(), which may be a problem for SMP system with non-per-CPU event timers, where that lock is global. I remember there were cases of 8-core systems using HPET by default, where this lock was pretty busy.

Mar 24 2018, 4:14 PM

Mar 23 2018

mav committed rS331414: Reduce struct aggsum_bucket padding to fit into one cache line..
Reduce struct aggsum_bucket padding to fit into one cache line.
Mar 23 2018, 2:51 AM
mav committed rS331408: MFV r331407: 9213 zfs: sytem typo.
MFV r331407: 9213 zfs: sytem typo
Mar 23 2018, 2:30 AM
mav committed rS331407: 9213 zfs: sytem typo.
9213 zfs: sytem typo
Mar 23 2018, 2:28 AM
mav committed rS331406: MFV r331405: 9084 spa_*_ashift must ignore spare devices.
MFV r331405: 9084 spa_*_ashift must ignore spare devices
Mar 23 2018, 2:25 AM
mav committed rS331405: 9084 spa_*_ashift must ignore spare devices.
9084 spa_*_ashift must ignore spare devices
Mar 23 2018, 2:22 AM
mav committed rS331404: MFV r331400: 8484 Implement aggregate sum and use for arc counters.
MFV r331400: 8484 Implement aggregate sum and use for arc counters
Mar 23 2018, 2:15 AM
mav committed rS331400: 8484 Implement aggregate sum and use for arc counters.
8484 Implement aggregate sum and use for arc counters
Mar 23 2018, 12:21 AM

Mar 22 2018

mav committed rS331399: MFC r329694: MFV r324198: 8081 Compiler warnings in zdb.
MFC r329694: MFV r324198: 8081 Compiler warnings in zdb
Mar 22 2018, 11:58 PM
mav committed rS331398: MFC r329691: MFV r322231:.
MFC r329691: MFV r322231:
Mar 22 2018, 11:57 PM
mav committed rS331397: MFC r329690: MFV r319737: 6939 add sysevents to zfs core for commands.
MFC r329690: MFV r319737: 6939 add sysevents to zfs core for commands
Mar 22 2018, 11:56 PM
mav committed rS331396: MFC r329683: MFV r319736: 6396 remove SVM.
MFC r329683: MFV r319736: 6396 remove SVM
Mar 22 2018, 11:55 PM