From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Subject: Re: [PATCH 3.12 16/33] isdn/gigaset: reset tty->receive_room when
 attaching ser_gigaset
To: Peter Hurley <peter@hurleysoftware.com>
References: <cover.1442326825.git.jslaby@suse.cz>
 <350624fa32cb152bfec51f236b0b62b8d480a05a.1442326825.git.jslaby@suse.cz>
 <4a187391f4c7d54b9c07eed08de48ffd6e0a3f20.1442326825.git.jslaby@suse.cz>
 <CAKU3ayUyyYZzDr-AiZEW7uy=kwTtNptnOZMAKEYyn+=dd_47bg@mail.gmail.com>
Cc: Jiri Slaby <jslaby@suse.cz>,
	"stable@vger.kernel.org" <stable@vger.kernel.org>,
	Linux kernel mailing list <linux-kernel@vger.kernel.org>,
	"David S. Miller" <davem@davemloft.net>
From: Tilman Schmidt <tilman@imap.cc>
Message-ID: <55F8B9D9.5060201@imap.cc>
Date: Wed, 16 Sep 2015 02:37:45 +0200
MIME-Version: 1.0
In-Reply-To: <CAKU3ayUyyYZzDr-AiZEW7uy=kwTtNptnOZMAKEYyn+=dd_47bg@mail.gmail.com>
Content-Type: multipart/signed; micalg=pgp-sha256;
 protocol="application/pgp-signature";
 boundary="CLAOi0VDA2JAXuDtJSr4WTUGH4bfNLWBJ"
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <stable.vger.kernel.org>

This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--CLAOi0VDA2JAXuDtJSr4WTUGH4bfNLWBJ
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

Am 16.09.2015 um 01:08 schrieb Peter Hurley:
> On Tue, Sep 15, 2015 at 10:22 AM, Jiri Slaby <jslaby@suse.cz
> <mailto:jslaby@suse.cz>> wrote:
>=20
>     From: Tilman Schmidt <tilman@imap.cc>
>=20
>     3.12-stable review patch.  If anyone has any objections, please let=

>     me know.
>=20
>     =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>=20
>     [ Upstream commit fd98e9419d8d622a4de91f76b306af6aa627aa9c ]
>=20
>     Commit 79901317ce80 ("n_tty: Don't flush buffer when closing ldisc"=
),
>     first merged in kernel release 3.10, caused the following regressio=
n
>     in the Gigaset M101 driver:
>=20
>=20
> Again, I'll just note my objection to this commit log.
>=20
> This driver was always broken because it never initialized
> tty->receive_room,
> but rather relied on common but not guaranteed circumstances to
> function.
>=20
> The commit noted simply made the underlying bug more evident, but the
> root cause was from the original merge commit of this driver.

I must admit I still don't understand that objection. The meaning of the
term "regression" is simply that something which previously worked
stopped working. It doesn't imply any statement about the root cause.

The ser-gigaset driver worked before the introduction of commit
79901317ce80. It didn't work anymore after the introduction of that
commit. So it is correct, and does not contradict your statements above
in any way, to state that commit introduced the described regression.

--=20
Tilman Schmidt                              E-Mail: tilman@imap.cc
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Unge=C3=B6ffnet mindestens haltbar bis: (siehe R=C3=BCckseite)


--CLAOi0VDA2JAXuDtJSr4WTUGH4bfNLWBJ
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQEcBAEBCAAGBQJV+LnjAAoJEFPuqx0v+F+qGyMIALYsz3BO0teqctiI2OPj8ETT
NpPQtat+/OTNYkbeaZ37KzOqNzemgWJPBOBuk0cu73VK3NU9jD1C2uOtqVGmw69+
rwzb8Pd9S5eE9WgdKOB8tnodW4RwiZscOMUmPkTj37BQqdbvNVqP5/rilsie8M0+
QVMDt/TsVMJQpCtvyL159fOrip8MToZUiLCzgR71BsRr6rfFh3xISDUkQm1tz/4Z
VkGGXsXv7TBZM/DvcXQzf+4H6wFgweOK6yHo2lm32heJhCU1MISKHzBzttGvQzpZ
fh2Fj6orJyCWeMAMDfYJwiG5jGyc36ZHKxwtVJXltk3lgDHkZteYkFJsngIi6kg=
=aBFj
-----END PGP SIGNATURE-----

--CLAOi0VDA2JAXuDtJSr4WTUGH4bfNLWBJ--