Messages in this thread | | | Date | Thu, 26 Jan 2023 12:19:57 +0100 | From | Jules Maselbas <> | Subject | Re: [RFC PATCH v2 11/31] kvx: Add atomic/locking headers |
| |
On Thu, Jan 26, 2023 at 10:57:20AM +0100, Jules Maselbas wrote: > Hi Mark, ...
> > > +static inline int arch_atomic_add_return(int i, atomic_t *v) > > > +{ > > > + int new, old, ret; > > > + > > > + do { > > > + old = v->counter; > > > > Likewise, arch_atomic64_read(v) here. > ack, this will bt arch_atomic_read(v) here since this is not atomic64_t > here. I took a second look at this and I think we are not doing the right thing, we do not need to defined arch_atomic_add_return at all since we are including the generic atomic right after, which will define the macro arch_atomic_add_return as generic_atomic_add_return
> > Thanks > -- Jules > > > > > > > > >
| |