Re: [PATCH] PCI: Restrict VPD read permission to root

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

 



Hi Jesse,

I saw Matthew's comment, and he is correct about the revision field
being populated.  I am regenerating and testing the updated patch now,
and will get it to you in about 15-20 minutes.  Thanks again.

-Ben

On Wed, 2008-07-02 at 10:49 -0700, Jesse Barnes wrote:
> On Wednesday, July 02, 2008 10:11 am Benjamin Li wrote:
> > Hi Ben,
> >
> > Currently what we are seeing on older RHEL 4 and 5 machine is that udev
> > is looking through the entire '/sys/bus/pci/devices/...' directory and
> > reading the vpd contents.  When this happens our devices are hung before
> > any user gets a chance to log in.  Therefore, we need to prevent this
> > from happening at boot time.  Changing the permissions to root didn't
> > solve our problem because we are running udev with root permissions.
> >
> > Thanks again.
> 
> Yeah I remember dealing with something similar when we added the sysfs rom 
> file.  Why HAL opens & reads every sysfs file boggles the mind, but old 
> versions do...
> 
> I'll queue up the fix and send another pull request to Linus today; did you 
> see Matthew's comment about the cached revision field?
> 
> Thanks,
> Jesse
> 

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