Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2013-07-23 Thread Marek Olšák
FYI, OpenGL 4.4, which was released yesterday, adds GL_MAP_PERSISTENT and GL_MAP_COHERENT bits as valid parameters of glMapBufferRange and glBufferStorage, allowing to use buffers for rendering while they are mapped and upload/download data to/from the buffers simultaneously. It's now clear that

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2013-07-23 Thread Jose Fonseca
- Original Message - FYI, OpenGL 4.4, which was released yesterday, adds GL_MAP_PERSISTENT and GL_MAP_COHERENT bits as valid parameters of glMapBufferRange and glBufferStorage, allowing to use buffers for rendering while they are mapped and upload/download data to/from the buffers

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2013-07-23 Thread Marek Olšák
Yes, absolutely, we should have a CAP for that. Marek On Wed, Jul 24, 2013 at 12:20 AM, Jose Fonseca jfons...@vmware.com wrote: - Original Message - FYI, OpenGL 4.4, which was released yesterday, adds GL_MAP_PERSISTENT and GL_MAP_COHERENT bits as valid parameters of glMapBufferRange

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-27 Thread Michel Dänzer
On Don, 2012-01-26 at 20:45 +0100, Marek Olšák wrote: On Tue, Jan 10, 2012 at 6:20 PM, Jose Fonseca jfons...@vmware.com wrote: On Tue, Jan 10, 2012 at 5:15 PM, Jose Fonseca jfons...@vmware.com wrote: Also, please provide app name and performance figures w/ this change. OK.

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-26 Thread Marek Olšák
On Tue, Jan 10, 2012 at 6:20 PM, Jose Fonseca jfons...@vmware.com wrote: - Original Message - On Tue, Jan 10, 2012 at 5:15 PM, Jose Fonseca jfons...@vmware.com wrote: - Original Message - The flag is optional, it doesn't have to implemented by everybody. If we do the

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
Still catching up on email traffic during holidays... I agree that user buffer uploads should be moved out of drivers, but I don't think this is the way to go. This PIPE_TRANSFER_MAP_PERMANENTLY means the driver relinquishes the ability to transform this data in any way before reashing the

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Christoph Bumiller
On 10.01.2012 12:29, Jose Fonseca wrote: Still catching up on email traffic during holidays... I agree that user buffer uploads should be moved out of drivers, but I don't think this is the way to go. I don't. If the state tracker uploads user buffers and presents them to the driver as

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
- Original Message - On 10.01.2012 12:29, Jose Fonseca wrote: Still catching up on email traffic during holidays... I agree that user buffer uploads should be moved out of drivers, but I don't think this is the way to go. I don't. If the state tracker uploads user buffers and

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Marek Olšák
On Tue, Jan 10, 2012 at 12:29 PM, Jose Fonseca jfons...@vmware.com wrote: Still catching up on email traffic during holidays... I agree that user buffer uploads should be moved out of drivers, but I don't think this is the way to go. This PIPE_TRANSFER_MAP_PERMANENTLY means the driver

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
- Original Message - On Tue, Jan 10, 2012 at 12:29 PM, Jose Fonseca jfons...@vmware.com wrote: Still catching up on email traffic during holidays... I agree that user buffer uploads should be moved out of drivers, but I don't think this is the way to go. This

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Christoph Bumiller
On 01/10/2012 01:13 PM, Jose Fonseca wrote: - Original Message - On 10.01.2012 12:29, Jose Fonseca wrote: Still catching up on email traffic during holidays... I agree that user buffer uploads should be moved out of drivers, but I don't think this is the way to go. I don't. If the

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
- Original Message - On 01/10/2012 01:13 PM, Jose Fonseca wrote: - Original Message - On 10.01.2012 12:29, Jose Fonseca wrote: Still catching up on email traffic during holidays... I agree that user buffer uploads should be moved out of drivers, but I don't think

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Marek Olšák
On Tue, Jan 10, 2012 at 5:15 PM, Jose Fonseca jfons...@vmware.com wrote: - Original Message - The flag is optional, it doesn't have to implemented by everybody. If we do the uploads in the state tracker, we will also do any required data transformation so that drivers don't have to do

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-10 Thread Jose Fonseca
- Original Message - On Tue, Jan 10, 2012 at 5:15 PM, Jose Fonseca jfons...@vmware.com wrote: - Original Message - The flag is optional, it doesn't have to implemented by everybody. If we do the uploads in the state tracker, we will also do any required data

Re: [Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-02 Thread Christian König
Looks good on first sign. I'm waiting for this for quite some time now, cause it makes XvMC state tracker implementation more cleaner and faster. Acked-by: Christian König deathsim...@vodafone.de On 02.01.2012 01:22, Marek Olšák wrote: Please see the diff for further info. This paves the way

[Mesa-dev] [PATCH 2/2] gallium: add flag PIPE_TRANSFER_MAP_PERMANENTLY

2012-01-01 Thread Marek Olšák
Please see the diff for further info. This paves the way for moving user buffer uploads out of drivers and should allow to clean up the mess in u_upload_mgr in the meantime. For now only allowed for buffers on r300 and r600. --- src/gallium/drivers/i915/i915_resource_buffer.c |7 ++-