linux-kernel - Re: [RFC] full suspend/resume support for i915 DRM driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <200710240818.31128.jbarnes@virtuousgeek.org>
Date:	Wed, 24 Oct 2007 08:18:30 -0700
From:	Jesse Barnes <jbarnes@...tuousgeek.org>
To:	Pavel Machek <pavel@....cz>
Cc:	dri-devel@...ts.sf.net, linux-kernel@...r.kernel.org
Subject: Re: [RFC] full suspend/resume support for i915 DRM driver

On Wednesday, October 24, 2007 6:35:14 am Pavel Machek wrote:
> Hi!
>
> > We seem to see a lot of bug reports along the lines of, "my machine
> > resumes but I can't see X" or, "I can see X but only with a bright
> > flashlight", etc.  These sorts of problems are due to the fact that
> > the X server isn't designed to do full state save/restore, and none
> > of the available kernel drivers do it on its behalf.
> >
> > Since intelfb and the rest of the Intel drivers are fairly incompatible,
> > this patch makes the DRM bind to the PCI device so it can register real
> > suspend/resume handlers.  Those handlers take care of saving and
> > restoring enough state for X to come back reliably on at least one of my
> > problematic test machines, but text mode still has trouble (still
> > debugging VGA state save/restore, including trying to save/restore
> > actual VRAM contents for possible hibernate support).
> >
> > How does this approach look?  Is a new DRM driver flag a good thing for
> > similar situations with other drivers?  Thoughts?
>
> Looks okay to me... from very quick look.
>
> > +	if (!i915_pipe_enabled(dev, pipe))
> > +		return;
> > +
> > +	if (pipe == PIPE_A)
> > +		array = dev_priv->savePaletteA;
>
> coding style, we probably want save_palette_A.

Yeah, I tried not to pull over uglies from the X code but I guess I forgot 
this bit.  I should also update the copyright.

> > +	unsigned long reg = pipe == PIPE_A ? PALETTE_A : PALETTE_B;
>
> Uff. Mixing = and == and ? in one expression is evil.

I could put parens around it if you think that would help, or just move it to 
a new line...

> I think I seen some #if 0 code.... just remove that.

Oh I left some in there?  Yeah I'll remove it.

Thanks,
Jesse
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/