Re: [RESEND][PATCH] PCI: Reassign page-aligned memory resources to device for pci passthrough.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Monday, December 08, 2008 11:57 pm Yuji Shimada wrote:
> On Fri, 5 Dec 2008 10:08:54 -0800
>
> Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> wrote:
> > On Thursday, December 4, 2008 12:43 am Yuji Shimada wrote:
> > > What do you think about supporting both boottime and runtime
> > > re-assignment.
> > >
> > > I find we can re-assign resources using fakephp driver. What I need to
> > > do is enhancing my patch to set "pagealignmem" parameter at runtime.
> > >
> > > I think it is good to create "/proc/bus/pci/pagealignmem".
> > >
> > > Re-assignment can be done as follows.
> > >
> > > 1. Sets SSSS:DD:BB.F of the device to "pagealignmem" parameter at
> > > runtime. 2. Hot-remove the device
> > > 3. Hot-add it.
> > >
> > >
> > > If you agree with me, I will submit new patch.
> >
> > This is a step in the right direction, thanks for pursuing it.
>
> Thanks too.
>
> > It might be better to put the pagealignmem file (and maybe just call it
> > "resource_alignment") into the sysfs directory for the device instead
> > though, then we could write various alignment constraints to it and hot
> > plug as you suggest.
>
> I can't put the "resource_alignment" file into sysfs directory for the
> device like "/sys/bus/pci/device/SSSS:BB:DD.F", because it disappears
> when device is hot-removed.

Oh yeah, of course. :)

> How about setting ALIGN@SSSS:BB:DD.F to "/sys/bus/pci/resource_alignment"?
> ALIGN means alignment for corresponding device.

Yeah, that sounds better; and it keeps things out of /proc which is good.  
Maybe other people have thoughts about it...  Greg may object to having a 
'resource_alignment' file that might possibly contain multiple {device, 
alignment} pairs though.

-- 
Jesse Barnes, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux