Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver

On Thu, 2006-03-16 at 17:46 +0000, Hugh Dickins wrote:

> > >   It seems one should do
> > > get_page() in .nopage, and then the driver can do dma_free_coherent()
> > > when the vma is released.
> > 
> > If that were the case, I'm unclear on how I would do this.  Add a
> > vmops->close method along with the existing vmops->nopage handler?
> 
> If neither mmap nor nopage allocated something, then vmops->close
> would be the wrong point at which to free it, I think.

I think I've satisfied myself that what we're now doing is fairly sane,
so I think we're OK.  I'm sure I'll be corrected in the next round of
driver submission patches if I'm wrong :-)

	<b

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: Roland Dreier <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: "Bryan O'Sullivan" <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: Roland Dreier <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: "Bryan O'Sullivan" <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: "Bryan O'Sullivan" <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: Roland Dreier <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: "Bryan O'Sullivan" <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: Roland Dreier <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: "Bryan O'Sullivan" <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: "Bryan O'Sullivan" <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: Roland Dreier <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: "Bryan O'Sullivan" <[email protected]>
  • Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
    • From: Hugh Dickins <[email protected]>

• Prev by Date: Re: 2.6.16-rc6-rt7
• Next by Date: Re: Router stops routing after changing MAC Address
• Previous by thread: Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
• Next by thread: Re: [PATCH 10 of 20] ipath - support for userspace apps using core driver
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]