Messages in this thread | | | Subject | Re: [PATCH] Suspend2 Merge: Driver model patches 2/2 | From | Nigel Cunningham <> | Date | Fri, 17 Sep 2004 09:19:53 +1000 |
| |
Hi.
On Fri, 2004-09-17 at 09:07, Greg KH wrote: > > What's broken? (I want to learn what I've done wrong that I'm not > > seeing). > > - No locking when traversing the list. > - Reference count needs to be bumped before returning a pointer to the > object you found.
Ah. Fair enough. I haven't seen any problems because the locking is more abstract: processes are frozen when this runs for suspend. I'll fix it, but won't bother resubmitting it because Pavel's changes should obsolete this stuff.
> > > mention the fact that the functionality this function proposes to offer > > > is not needed either. > > > > > > > (their patch just fills in a field that was left blank previously), > > > > > > What patch? > > > > Attached. Sorry if I wrongly assumed this was the patch you're talking > > about. > > Ah, no, I've never seen this one, thanks. But it looks sane, I don't > have a problem with it (sysfs will like it, it's not a suspend specific > patch at all.)
Antonio posted it to LKML last week IIRC, which is why I didn't include it in the device driver patches. Given Pavel's changes (again), I'm in two minds as to whether its needed. It's clearly the right thing to do, but not needed at the moment. Then again, as we noted already, the whole device_class thing doesn't get a lot of use at the moment.
Regards,
Nigel -- Nigel Cunningham Pastoral Worker Christian Reformed Church of Tuggeranong PO Box 1004, Tuggeranong, ACT 2901
Many today claim to be tolerant. True tolerance, however, can cope with others being intolerant.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |