Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-02-03 Thread Tvrtko Ursulin
On 02/02/2023 23:43, T.J. Mercier wrote: On Wed, Feb 1, 2023 at 6:23 AM Tvrtko Ursulin wrote: On 01/02/2023 01:49, T.J. Mercier wrote: On Tue, Jan 31, 2023 at 6:01 AM Tvrtko Ursulin wrote: On 25/01/2023 20:04, T.J. Mercier wrote: On Wed, Jan 25, 2023 at 9:31 AM Tvrtko Ursulin wrote:

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-02-03 Thread Tvrtko Ursulin
On 02/02/2023 23:43, T.J. Mercier wrote: On Wed, Feb 1, 2023 at 6:52 AM Tvrtko Ursulin wrote: On 01/02/2023 14:23, Tvrtko Ursulin wrote: On 01/02/2023 01:49, T.J. Mercier wrote: On Tue, Jan 31, 2023 at 6:01 AM Tvrtko Ursulin wrote: On 25/01/2023 20:04, T.J. Mercier wrote: On Wed, Ja

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-02-02 Thread T.J. Mercier
On Wed, Feb 1, 2023 at 6:52 AM Tvrtko Ursulin wrote: > > > On 01/02/2023 14:23, Tvrtko Ursulin wrote: > > > > On 01/02/2023 01:49, T.J. Mercier wrote: > >> On Tue, Jan 31, 2023 at 6:01 AM Tvrtko Ursulin > >> wrote: > >>> > >>> > >>> On 25/01/2023 20:04, T.J. Mercier wrote: > On Wed, Jan 25,

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-02-02 Thread T.J. Mercier
On Wed, Feb 1, 2023 at 6:23 AM Tvrtko Ursulin wrote: > > > On 01/02/2023 01:49, T.J. Mercier wrote: > > On Tue, Jan 31, 2023 at 6:01 AM Tvrtko Ursulin > > wrote: > >> > >> > >> On 25/01/2023 20:04, T.J. Mercier wrote: > >>> On Wed, Jan 25, 2023 at 9:31 AM Tvrtko Ursulin > >>> wrote: > > >>>

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-02-01 Thread Tvrtko Ursulin
On 01/02/2023 14:23, Tvrtko Ursulin wrote: On 01/02/2023 01:49, T.J. Mercier wrote: On Tue, Jan 31, 2023 at 6:01 AM Tvrtko Ursulin wrote: On 25/01/2023 20:04, T.J. Mercier wrote: On Wed, Jan 25, 2023 at 9:31 AM Tvrtko Ursulin wrote: Hi, On 25/01/2023 11:52, Michal Hocko wrote: On T

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-02-01 Thread Tvrtko Ursulin
On 01/02/2023 01:49, T.J. Mercier wrote: On Tue, Jan 31, 2023 at 6:01 AM Tvrtko Ursulin wrote: On 25/01/2023 20:04, T.J. Mercier wrote: On Wed, Jan 25, 2023 at 9:31 AM Tvrtko Ursulin wrote: Hi, On 25/01/2023 11:52, Michal Hocko wrote: On Tue 24-01-23 19:46:28, Shakeel Butt wrote: On

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-31 Thread T.J. Mercier
On Tue, Jan 31, 2023 at 6:01 AM Tvrtko Ursulin wrote: > > > On 25/01/2023 20:04, T.J. Mercier wrote: > > On Wed, Jan 25, 2023 at 9:31 AM Tvrtko Ursulin > > wrote: > >> > >> > >> Hi, > >> > >> On 25/01/2023 11:52, Michal Hocko wrote: > >>> On Tue 24-01-23 19:46:28, Shakeel Butt wrote: > On Tu

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-31 Thread Tvrtko Ursulin
On 25/01/2023 20:04, T.J. Mercier wrote: On Wed, Jan 25, 2023 at 9:31 AM Tvrtko Ursulin wrote: Hi, On 25/01/2023 11:52, Michal Hocko wrote: On Tue 24-01-23 19:46:28, Shakeel Butt wrote: On Tue, Jan 24, 2023 at 03:59:58PM +0100, Michal Hocko wrote: On Mon 23-01-23 19:17:23, T.J. Mercier

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-25 Thread T.J. Mercier
On Wed, Jan 25, 2023 at 4:05 AM Michal Hocko wrote: > > On Tue 24-01-23 10:55:21, T.J. Mercier wrote: > > On Tue, Jan 24, 2023 at 7:00 AM Michal Hocko wrote: > > > > > > On Mon 23-01-23 19:17:23, T.J. Mercier wrote: > > > > When a buffer is exported to userspace, use memcg to attribute the > > >

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-25 Thread T.J. Mercier
On Wed, Jan 25, 2023 at 9:31 AM Tvrtko Ursulin wrote: > > > Hi, > > On 25/01/2023 11:52, Michal Hocko wrote: > > On Tue 24-01-23 19:46:28, Shakeel Butt wrote: > >> On Tue, Jan 24, 2023 at 03:59:58PM +0100, Michal Hocko wrote: > >>> On Mon 23-01-23 19:17:23, T.J. Mercier wrote: > When a buffer

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-25 Thread Tvrtko Ursulin
Hi, On 25/01/2023 11:52, Michal Hocko wrote: On Tue 24-01-23 19:46:28, Shakeel Butt wrote: On Tue, Jan 24, 2023 at 03:59:58PM +0100, Michal Hocko wrote: On Mon 23-01-23 19:17:23, T.J. Mercier wrote: When a buffer is exported to userspace, use memcg to attribute the buffer to the allocating

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-25 Thread Michal Hocko
On Tue 24-01-23 10:55:21, T.J. Mercier wrote: > On Tue, Jan 24, 2023 at 7:00 AM Michal Hocko wrote: > > > > On Mon 23-01-23 19:17:23, T.J. Mercier wrote: > > > When a buffer is exported to userspace, use memcg to attribute the > > > buffer to the allocating cgroup until all buffer references are >

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-25 Thread Michal Hocko
On Tue 24-01-23 19:46:28, Shakeel Butt wrote: > On Tue, Jan 24, 2023 at 03:59:58PM +0100, Michal Hocko wrote: > > On Mon 23-01-23 19:17:23, T.J. Mercier wrote: > > > When a buffer is exported to userspace, use memcg to attribute the > > > buffer to the allocating cgroup until all buffer references

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-24 Thread Shakeel Butt
On Tue, Jan 24, 2023 at 03:59:58PM +0100, Michal Hocko wrote: > On Mon 23-01-23 19:17:23, T.J. Mercier wrote: > > When a buffer is exported to userspace, use memcg to attribute the > > buffer to the allocating cgroup until all buffer references are > > released. > > Is there any reason why this me

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-24 Thread T.J. Mercier
On Tue, Jan 24, 2023 at 7:00 AM Michal Hocko wrote: > > On Mon 23-01-23 19:17:23, T.J. Mercier wrote: > > When a buffer is exported to userspace, use memcg to attribute the > > buffer to the allocating cgroup until all buffer references are > > released. > > Is there any reason why this memory can

Re: [PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-24 Thread Michal Hocko
On Mon 23-01-23 19:17:23, T.J. Mercier wrote: > When a buffer is exported to userspace, use memcg to attribute the > buffer to the allocating cgroup until all buffer references are > released. Is there any reason why this memory cannot be charged during the allocation (__GFP_ACCOUNT used)? Also yo

[PATCH v2 1/4] memcg: Track exported dma-buffers

2023-01-23 Thread T.J. Mercier
When a buffer is exported to userspace, use memcg to attribute the buffer to the allocating cgroup until all buffer references are released. Unlike the dmabuf sysfs stats implementation, this memcg accounting avoids contention over the kernfs_rwsem incurred when creating or removing nodes. Signed