Re: [PATCH] Assign memory resource with specified alignment.

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

 



On Mon, 2 Mar 2009 12:15:20 -0700
Alex Chiang <achiang@xxxxxx> wrote:

> * Yuji Shimada <shimada-yxb@xxxxxxxxxxxxxxx>:
> > On Tue, 24 Feb 2009 09:24:13 -0800
> > Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> wrote:
> > 
> > > On Tuesday, February 17, 2009 2:19:39 am Yuji Shimada wrote:
> > > > > If resource_alignment is a per-device attribute, doesn't it make
> > > > > sense to put the file in /sys/bus/pci/devices/resource_alignment ?
> > > >
> > > > I don't think it is easy to add "/sys/bus/pci/devices/resource_alignment".
> > > >
> > > > In case of "/sys/bus/pci/resource_alignment", we can use BUS_ATTR.
> > > > In case of "/sys/bus/pci/devices/SSSS:BB:DD.F/resource_alignment", we
> > > > can use DEVICE_ATTR.
> > > >
> > > > But in case of "/sys/bus/pci/devices/resource_alignment", what can we
> > > > use?
> > > 
> > > Yeah, might be a little trickier.  Really it's a property of the bus scan & 
> > > resource allocation, so another option would be (with Alex's new stuff) to 
> > > make it part of the actual rescan command somehow.
> > 
> > "rescan" file will be added to "/sys/bus/pci".
> > If I add "resource_alignment" file to "/sys/bus/pci", both of
> > them will be in the same directory.
> > I think this is reasonable.
> > 
> > I'd like to add "/sys/bus/pci/resource_alignment" file.
> 
> One thing that wasn't clear to me -- can I specify one alignment
> for one device and a different alignment for a different device?

Yes. You can specify them as follows.

     20@07:00.0;18@00:1d.7

Thanks,
--
Yuji Shimada

--
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