Re: Resolving physical addresses (change in 2.4.x?)

Re: Resolving physical addresses (change in 2.4.x?)

[Pete Zaitcev]

> Even recognizing these as unmapped and resolving anew, produced a frozen
> machine once the DMA used these addresses. Was there a change in 2.4.x so
> that my resolving routine now works incorrect?
> 
> /*resolve virt. addresses to phys.*/
> unsigned long ch_get_physpage(unsigned long virtaddr)
> {
>   /*Stuff for browsing through the memory page tables*/
>   pgd_t *pgd_t_dir;
>   pmd_t *pmd_t_dir;
>   pte_t *pte_t_dir;
> 
>   /*Get physical address*/
>   pgd_t_dir=pgd_offset(current->mm,virtaddr);
>   pmd_t_dir=pmd_offset(pgd_t_dir,virtaddr);
>   pte_t_dir=pte_offset(pmd_t_dir,virtaddr);
>   return virt_to_bus((void *)pte_page(*pte_t_dir));
> }

Are you crazy? This routing could NEVER work; pge_page returns
a pointer to a struct page. I always was suspicious of this
technique; even if you find out how to use page_address,
what do you do about the bus address? On some architectures,
the bus address is only known to the part that manages a
north bridge; it may even be stored in hardware registers.
No matter how popular this trick is, it is highly illegal.

To work properly, your driver has to remember virtual and bus
addresses that were mapped with pci_alloc_sg. They are returned to
you for this very purpose. In most cases you have to to track I/Os 
anyway, so just add a field to whatever management structure you use.

-- Pete

Resolving physical addresses (change in 2.4.x?)

[Christoph Baumann]

Hi,

with a 2.2.19 kernel I used the routine below to resolve physical addresses
for preparing gather/scatter DMA. Using it on unmapped memory (allocated but
not yet used) it returned zero. Knowing this I could force the mapping of
such memory. Now with 2.4.x kernels I get something like 0x100000 for
unmapped pages.
Even recognizing these as unmapped and resolving anew, produced a frozen
machine once the DMA used these addresses. Was there a change in 2.4.x so
that my resolving routine now works incorrect?

/*resolve virt. addresses to phys.*/
unsigned long ch_get_physpage(unsigned long virtaddr)
{
  /*Stuff for browsing through the memory page tables*/
  pgd_t *pgd_t_dir;
  pmd_t *pmd_t_dir;
  pte_t *pte_t_dir;

  /*Get physical address*/
  pgd_t_dir=pgd_offset(current->mm,virtaddr);
  pmd_t_dir=pmd_offset(pgd_t_dir,virtaddr);
  pte_t_dir=pte_offset(pmd_t_dir,virtaddr);
  return virt_to_bus((void *)pte_page(*pte_t_dir));
}


Mit freundlichen Gruessen / Best regards
Dipl.-Phys. Christoph Baumann
---
SORCUS Computer GmbH
Im Breitspiel 11 c
D-69126 Heidelberg

Tel.: +49(0)6221/3206-0
Fax: +49(0)6221/3206-66