Re: [patch] threading fix, tid-2.5.47-A3

[Luca Barbieri]

--=-ZT9r2MPz6LPXviQcwJRl
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable

> It's all encapsulated in the libpthread which is used.  No apps need to
> be recompiled so it is OK to make this incompatible change.
My point was that aesthetical reasons do not justify breaking anything
(aka forcing people to figure out what happened and spend time
recompiling).
Anyway, I'll need to recompile anyway, so I don't really care.

> I cannot see any reasonable way out if any of the put_user calls fail?
> Do you want the clone() call to fail if the parent's receiving address
> is wrong?  You'd have to go and kill the child again since it's already
> created.
>=20
> Instead let the user initialize the memory location the clone call is
> supposed to write in with zero.  if the value didn't change the
> user-level code can detect the error and handle appropriately.  So,
> ignore the put_user errors.  Maybe say so explicitly and add (void) in
> front.
I proposed to fail in put_user for the parent tid simply because the old
code did it.
I'm not completely sure whether we should fail or not, but if put_user
fails something bad is happening, so it may be better to signal the
error rather than just continuing since it can be done easily (for the
parent tid).

--=-ZT9r2MPz6LPXviQcwJRl
Content-Type: application/pgp-signature; name=signature.asc
Content-Description: This is a digitally signed message part

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQA92AC/djkty3ft5+cRAvW0AJwL7SIk4gkBqQps8htdNFrD2SATxwCeOuYU
wgdiMGaYvGYwDxmZwV3hejo=
=WsLR
-----END PGP SIGNATURE-----

--=-ZT9r2MPz6LPXviQcwJRl--