Page MenuHomeFreeBSD

multimediaUmbrella
ActivePublic

Recent Activity

Oct 6 2018

tcberner added a member for multimedia: tcberner.
Oct 6 2018, 11:31 PM

Dec 12 2017

madpilot closed D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.
Dec 12 2017, 9:30 PM · x11, multimedia

Nov 11 2017

madpilot updated the diff for D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.

Update patch to ports tree r453973

Nov 11 2017, 2:23 PM · x11, multimedia

Nov 7 2017

madpilot updated the diff for D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.

Update diff to latest ports tree snapshot.

Nov 7 2017, 3:51 PM · x11, multimedia

Nov 5 2017

madpilot updated the diff for D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.

Removed wrong stanza modifying OPTIONS_DEFAULTS depending on external variable.

Nov 5 2017, 11:18 AM · x11, multimedia
madpilot planned changes to D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.
Nov 5 2017, 11:12 AM · x11, multimedia
mat added inline comments to D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.
Nov 5 2017, 10:55 AM · x11, multimedia

Oct 22 2017

cpm added a comment to D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.

Thanks for the heads up!

Oct 22 2017, 9:49 PM · x11, multimedia
cpm accepted D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.
Oct 22 2017, 9:45 PM · x11, multimedia
madpilot updated the diff for D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.

Updated patch.

Oct 22 2017, 9:44 PM · x11, multimedia
madpilot created D12761: libva update to 2.0.0, shlib bump to dependent ports and fix of two dependent ports.
Oct 22 2017, 9:39 PM · x11, multimedia

Jun 28 2017

jbeich added a watcher for multimedia: jbeich.
Jun 28 2017, 10:48 AM

Jun 10 2017

hselasky added a comment to D11140: OSS: allow unplug soundcars without apps close devices.

destroy_dev() will make sure all system calls have returned. In your patch are you destroying the device first?

Jun 10 2017, 9:36 PM · Contributor Reviews (base), multimedia
rozhuk.im-gmail.com added a comment to D11140: OSS: allow unplug soundcars without apps close devices.

Thank you.

Jun 10 2017, 9:18 PM · Contributor Reviews (base), multimedia
hselasky added a comment to D11140: OSS: allow unplug soundcars without apps close devices.

Make a test app like this:

Jun 10 2017, 9:10 PM · Contributor Reviews (base), multimedia
hselasky added a comment to D11140: OSS: allow unplug soundcars without apps close devices.

I'm afraid that pending read/write/ioctls might use freed structures. I'll try to analyze your patch carefully next week. Currently at bsdcan2017 .

Jun 10 2017, 9:07 PM · Contributor Reviews (base), multimedia
rozhuk.im-gmail.com added a comment to D11140: OSS: allow unplug soundcars without apps close devices.

No.
How I can check mem use after free and how turn on mem debug?

Jun 10 2017, 9:02 PM · Contributor Reviews (base), multimedia
hselasky added a comment to D11140: OSS: allow unplug soundcars without apps close devices.

Did you check that no memory is used after free?

Jun 10 2017, 8:55 PM · Contributor Reviews (base), multimedia
rozhuk.im-gmail.com created D11140: OSS: allow unplug soundcars without apps close devices.
Jun 10 2017, 8:50 PM · Contributor Reviews (base), multimedia

Dec 11 2016

kmacy abandoned D6620: pass fault type to vm_pager_get_pages.

Replaced by populate, which although it still insists on doing a great deal of accounting not needed for device mappings (COW cargo culting AFAICT) that complicates mapping the same device memory at different VAs - it is quite close to what is expected by graphics drivers. Thanks go to Kostik and Alan.

Dec 11 2016, 5:14 AM · multimedia
alc added a comment to D6620: pass fault type to vm_pager_get_pages.

Related change in rS309710, rS309712

Dec 11 2016, 5:00 AM · multimedia
emaste added a comment to D6620: pass fault type to vm_pager_get_pages.

Related change in rS309710, rS309712

Dec 11 2016, 1:27 AM · multimedia

May 31 2016

kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.

@kib To clarify: I only need 1 change to the VM and that is provided by your patch. I abandoned the other revision changing the device pager interface. Everything else is handled out of tree in the (soon to be) ports linuxkpi.

May 31 2016, 8:47 PM · multimedia
kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.
In D6620#140369, @kib wrote:
In D6620#140367, @kmacy wrote:

Why cannot you just install all pages into the object queue at once ?

May 31 2016, 8:29 PM · multimedia

May 30 2016

kib added a comment to D6620: pass fault type to vm_pager_get_pages.
In D6620#140367, @kmacy wrote:

Why cannot you just install all pages into the object queue at once ? I do not understand why do you want to also force-map them into userspace simultaneously. The difference would be in soft-faults indeed. But softfaults in vm_fault() would only result in looking up the page and installation of pte, without even touching the pager. More, I believe (but did not verified) that softfault would be handled by the vm_fault() 'fast path', which allows simultaneous faults handling (the object is read-locked).

May 30 2016, 8:34 PM · multimedia
kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.

This is the core of the i915_gem_fault routine. You will notice how it calls vm_insert_pfn for every page in the view - which in my testing can be as many as 1030+ pages.

May 30 2016, 7:59 PM · multimedia
kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.

If by aperture you mean the mappable portion of GTT, I in fact do not see why you statement is true. More, from what I remember from the times when I had to work on that code, it is in fact can be mapped more than once.

May 30 2016, 5:55 PM · multimedia
kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.
May 30 2016, 5:01 PM · multimedia
kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.
May 30 2016, 4:56 PM · multimedia
kib added a comment to D6620: pass fault type to vm_pager_get_pages.
In D6620#140319, @kmacy wrote:

Passing the mapping address is the huge layering violation. Digging the curproc map to find the faulting address is even larger one. What would happen if there is already another mapping in the same map for the same object ?

You would never have the same aperture mapped twice, and if you did it would not have any adverse effect. We really to need stop conflating device memory's behavior with RAM.

May 30 2016, 4:56 PM · multimedia
kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.

Passing the mapping address is the huge layering violation. Digging the curproc map to find the faulting address is even larger one. What would happen if there is already another mapping in the same map for the same object ?

You would never have the same aperture mapped twice, and if you did it would not have any adverse effect. We really to need stop conflating device memory's behavior with RAM.

May 30 2016, 4:33 PM · multimedia
kib added a comment to D6620: pass fault type to vm_pager_get_pages.
In D6620#140142, @kmacy wrote:

I can find the entry by searching the map for the object at fault time. This gives me the start of the mapping. I can then cache that start and cache the map address to look up a new entry if a second process maps the device. In the instances where the driver's fault routine returns more than one page I can simply call pmap_enter_object after all the pages have been added to the object.

In summary, the only ABI change I need to is to permit the fault routine to manage devp_pglist (along with passing down the actual fault type).

May 30 2016, 2:50 PM · multimedia
kmacy updated the diff for D6620: pass fault type to vm_pager_get_pages.

Switch to kib's patch, dropping the changes to VOP_GETPAGES

May 30 2016, 1:59 AM · multimedia
kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.

It looks like there's no clean way to work backwards from an object to a map_entry. Admittedly, it would be a bit of a layering violation and would cause problems for aliasing. Perhaps rather than pass in count and *rahead we can pass in the actual fault address?

May 30 2016, 12:16 AM · multimedia

May 29 2016

kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.

It occurs to me that if I can identify the virtual address that the object starts at I don't actually need to change the device pager ABI beyond adding a new OBJ type to indicate that the fault routine will manage the pglist itself. If the fault routine knows the actual base of where the object is being mapped it can insert the additional mappings itself. Additionally, it means the VM doesn't need to learn that COW is not relevant to device memory.

May 29 2016, 10:32 PM · multimedia
kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.
In D6620#140020, @kib wrote:

I revived my patch, it grown several mis-merges from the time of being written.

Main differences from yours is the use of VM_PROT_ALL for other places, and keeping VOP_GETPAGES() interface. I do not see it useful to pass fault type to VOP.

https://kib.kiev.ua/kib/cdev_prot.1.patch

May 29 2016, 7:29 PM · multimedia
kib added a comment to D6620: pass fault type to vm_pager_get_pages.

I revived my patch, it grown several mis-merges from the time of being written.

May 29 2016, 11:28 AM · multimedia
kmacy added a comment to D6620: pass fault type to vm_pager_get_pages.

This patch doesn't include the necessary update to md and tmpfs. If there's any chance of it going in as is I will update the patch.

May 29 2016, 12:40 AM · multimedia

May 28 2016

kmacy added inline comments to D6620: pass fault type to vm_pager_get_pages.
May 28 2016, 11:00 PM · multimedia
alc added inline comments to D6620: pass fault type to vm_pager_get_pages.
May 28 2016, 10:51 PM · multimedia
kmacy retitled D6620: pass fault type to vm_pager_get_pages from to pass fault type to vm_pager_get_pages.
May 28 2016, 10:40 PM · multimedia