Bug#604096: Bug#601341: Bug#602418: #601341, #602418 and #604096 seem to be duplicates

[Ian Campbell <ijc@hellion.org.uk>]

On Mon, 2010-12-20 at 11:42 -0500, Konrad Rzeszutek Wilk wrote:
> > Then I got to eba164ec7e69 "radeon/nouveau/ttm/AGP: Use dma_addr if TTM
> > has set it." which complained:
> >           CC [M]  drivers/gpu/drm/ttm/ttm_agp_backend.o
> >         drivers/gpu/drm/ttm/ttm_agp_backend.c: In function ‘ttm_agp_populate’:
> >         drivers/gpu/drm/ttm/ttm_agp_backend.c:66: error: ‘struct agp_memory’ has no member named ‘dma_addr’
> > and indeed the field is missing both in 2.6.32+drm33 and Linus' tree. Do
> > I need to cherry pick something from another series or is this commit
> 
> You can drop that patch. I've rebased the tree to:
> 
> devel/ttm.pci-api-v2
> 
> which is exactly like the older except missing that patch.

Thanks I saw the branch but didn't manage to actually spot the
difference.

> > I'll publish my backport in a git tree once I'm happy with it, I need to
> > tidy it up and correct the cherry-picked from comments etc and then
> > actually build something which uses it. I'll make Debian packages
> > available for wider testing once I've done that (with Xmas coming up I
> > don't know when that will actually be).

FWIW I ran a patched kernel up on my home machine (radeon) and it didn't
work. Without KMS the X server failed reasonably gracefuly (with some,
presumably spurious, message about the keyboard driver) and with KMS it
switched graphics mode and then hung on a black screen.

I'll keep poking but I'm hampered a bit by my only suitable "test
machine" actually being my home workstation.

I suppose I should poke through 2.6.33..2.6.37-rc and see if anything
jumps out for backporting.

> > Did the series make any waves upstream? What are the chances that it
> > will go upstream in something roughly like its current form?
> 
> I hope so. I am putting the polishing touches on item c) to have it ready for upstream.

Cool.

Hrm, do I need some equivalent of c) in order to have a chance of this
stuff working?

Ian.

-- 
Ian Campbell
Current Noise: Emperor - Grey

"You show me an American who can keep his mouth shut and I'll eat him."
		-- Newspaperman from Frank Capra's _Meet_John_Doe_