Looks very good!
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Aug 16 2019
Aug 15 2019
Aug 14 2019
Aug 13 2019
Looks good!
Aug 8 2019
@jhb : OK, if you can move lockdep_is_held() under #ifdef INVARIANTS and otherwise define lockdep_is_held() as 1, I think we are in-line.
Ok, can you tell me what the semantic in Linux is for lockdep_is_held() after on a lock locked via read_lock()?
If we look at Linux all the LOCKDEP stuff is under ifdef:
Aug 7 2019
Are we certain that lockep() is not used with read_lock()? Also I don't like that cast to lock_object. It would be better to do this clean by referring to the lock object field in the lock, assuming they are named then same. Or use WITNESS for selected locks, which is much better. The reason WITNESS is currently disabled for LinuxKPI spinlocks is that spinlocks are not destroyed under Linux, so the WITNESS object never goes away.
@hselasky Do you have any thoughts on the LOCK_CLASS-based approach? Is this something you can test? I've only tested it with drm.
Aug 6 2019
Please test!
Can you try this patch and report back?
Rebase patch after latest changes upstream.
Please test.
Can you test this patch?
Can you test this updated patch?
Aug 5 2019
Adding Slava to this review aswell.
Aug 4 2019
Looks good. Maybe you should also bump the __FreeBSD_version when you commit this.
Aug 2 2019
Aug 1 2019
There is no regular (operating on the integer regiter file) instructions on amd64 that could add e.g. three registers once. There is a LEA instruction which allows you to add to registers and a constant, but thats all.
I don't understand this comment. Can you elaborate? IMO, we want to differentiate from Linux when and only when it is possible to do better than Linux does; not gratuitously.
Maybe the following is unclear:
If the functions are not broken there is no reason to fix them from my POV.
Jul 31 2019
OK, I'll try to have a look at your patch.
Can you abandon this differential revision if no longer active?
Ping - is this patch still relevant?
Was patch updated?
Can this device be used via /usr/ports/multimedia/webcamd ?
Ping: Can this change be abandoned now?
Changes look good to me.
@jhibbits : Can you update the patch with the agreed upon changes?
Just remove lockdep_is_held(m) .
@zeising : Do you use CONFIG_LOCKDEP in drm-next ?
Jul 29 2019
Assume that the driver knows not to enable both MSIX and MSI.
Jul 28 2019
I'll have a closer look tomorrow. Most of this looks good.
Jul 26 2019
Looks good!
Jul 24 2019
Jul 21 2019
I'll have a closer look at you patch(es) when I'm back in office. Please stay tuned.
Jul 19 2019
What about lockedep_is_held() ??
Jul 18 2019
Jul 16 2019
Jul 15 2019
Jul 12 2019
Please stay tuned!
Jul 9 2019
Jul 8 2019
Please re-base this patch on top of the latest 13-current.
Jul 6 2019
Looks good. Some minor nits. Please fix before commit.
Jul 5 2019
Jul 4 2019
Jul 3 2019
Jul 2 2019
Patch looks good to me. Did you run with this patch?
Patch looks good to me. Did you run this patch, I.E. force the error case somehow?
Jun 29 2019
Can you upload full context for the diff?
Jun 28 2019
Spell lkpi -> LinuxKPI in the commit message.
Optimize epoch_drain_cb() by using an atomic operation.
Update manual page to describe new epoch(9) APIs.