From: Greg KH on
On Tue, Mar 02, 2010 at 06:55:20PM -0500, Ari Entlich wrote:
> Hello!
>
> Could you please remove this patch from your 2.6.34 tty pull request?
> Andrew Morton questioned whether this should actually be used yet[1],
> and I responded confirming his suspicions[2]. In addition, this patch
> has not "been in the linux-next and -mm releases for a number of
> weeks now", as you suggest in your pull request.

Hm, it's been there for over a week from what I can tell, right?

I'll revert it after this batch goes into Linus's tree. Next time, warn
me a bit earlier :)

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Ari Entlich on
----- "Greg KH" <gregkh(a)suse.de> wrote:
> Hm, it's been there for over a week from what I can tell, right?

Hmm yes, I suppose it has.

> I'll revert it after this batch goes into Linus's tree. Next time, warn
> me a bit earlier :)

I apologize; I would have if I had realized that you picking it up meant it
was on its way to mainline.

Ari
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Greg KH on
On Tue, Mar 02, 2010 at 07:16:08PM -0500, Ari Entlich wrote:
> ----- "Greg KH" <gregkh(a)suse.de> wrote:
> > Hm, it's been there for over a week from what I can tell, right?
>
> Hmm yes, I suppose it has.
>
> > I'll revert it after this batch goes into Linus's tree. Next time, warn
> > me a bit earlier :)
>
> I apologize; I would have if I had realized that you picking it up meant it
> was on its way to mainline.

So, are you happy with this in Linus's tree, or do you want it reverted?

thanks,
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Ari Entlich on
----- "Greg KH" <greg(a)kroah.com> wrote:
> So, are you happy with this in Linus's tree, or do you want it
> reverted?

The problems with this patch include:
1. There's at least one subtlety I overlooked - switching between X servers
(i.e. from one X VT to another) still requires the cooperation of both
X servers. I was assuming that KMS eliminated this.
2. It hasn't been tested at all (no X server patch exists which uses the
new mode).

So yes, I think it would be wise to revert it. I'll resubmit the patch (in
a changed form or otherwise) when I've resolved these issues.

I apologize again for being late in telling you this stuff.

Ari
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Greg KH on
On Mon, Mar 15, 2010 at 07:53:10PM -0400, Ari Entlich wrote:
> ----- "Greg KH" <greg(a)kroah.com> wrote:
> > So, are you happy with this in Linus's tree, or do you want it
> > reverted?
>
> The problems with this patch include:
> 1. There's at least one subtlety I overlooked - switching between X servers
> (i.e. from one X VT to another) still requires the cooperation of both
> X servers. I was assuming that KMS eliminated this.
> 2. It hasn't been tested at all (no X server patch exists which uses the
> new mode).
>
> So yes, I think it would be wise to revert it. I'll resubmit the patch (in
> a changed form or otherwise) when I've resolved these issues.
>
> I apologize again for being late in telling you this stuff.

No problem, I'll go queue up the revert.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/