Re: [PATCH v2 1/1] binder: fix freeze race

2021-09-13 Thread Li Li
On Fri, Sep 10, 2021 at 12:15 AM Greg KH wrote: > > On Thu, Sep 09, 2021 at 11:17:42PM -0700, Li Li wrote: > > On Thu, Sep 9, 2021 at 10:38 PM Greg KH wrote: > > > > > > On Thu, Sep 09, 2021 at 08:53:16PM -0700, Li Li wrote: > > > > struct binder_frozen_status_info { > > > > __u32

Re: [PATCH v2 1/1] binder: fix freeze race

2021-09-10 Thread Greg KH
On Thu, Sep 09, 2021 at 11:17:42PM -0700, Li Li wrote: > On Thu, Sep 9, 2021 at 10:38 PM Greg KH wrote: > > > > On Thu, Sep 09, 2021 at 08:53:16PM -0700, Li Li wrote: > > > struct binder_frozen_status_info { > > > __u32pid; > > > + > > > + /* process received sync transactio

Re: [PATCH v2 1/1] binder: fix freeze race

2021-09-09 Thread Li Li
On Thu, Sep 9, 2021 at 10:38 PM Greg KH wrote: > > On Thu, Sep 09, 2021 at 08:53:16PM -0700, Li Li wrote: > > struct binder_frozen_status_info { > > __u32pid; > > + > > + /* process received sync transactions since last frozen > > + * bit 0: received sync transaction af

Re: [PATCH v2 1/1] binder: fix freeze race

2021-09-09 Thread Greg KH
On Thu, Sep 09, 2021 at 08:53:16PM -0700, Li Li wrote: > From: Li Li > > Currently cgroup freezer is used to freeze the application threads, and > BINDER_FREEZE is used to freeze the corresponding binder interface. > There's already a mechanism in ioctl(BINDER_FREEZE) to wait for any > existing t

[PATCH v2 1/1] binder: fix freeze race

2021-09-09 Thread Li Li
From: Li Li Currently cgroup freezer is used to freeze the application threads, and BINDER_FREEZE is used to freeze the corresponding binder interface. There's already a mechanism in ioctl(BINDER_FREEZE) to wait for any existing transactions to drain out before actually freezing the binder interf