svn commit: r236935 - head/sys/kern
Pawel Jakub Dawidek
pjd at FreeBSD.org
Tue Jun 12 21:46:03 UTC 2012
On Tue, Jun 12, 2012 at 09:18:28PM +0200, Mateusz Guzik wrote:
> On Tue, Jun 12, 2012 at 06:01:29PM +0200, Pawel Jakub Dawidek wrote:
> > Look good and you can just add 'In co-operation with: pjd'.
> > One minor thing is that fd_first_free() can return 'size' if there are
> > no free slots available. Could you include that in the comment as well?
> >
>
> http://people.freebsd.org/~mjg/patches/fdalloc%2bfd_first_free2.patch
Ok. Merge racct_set() call into one line, it now fits into 80 chars.
I have no more objections.
> > > fd_last_used has very same problem with comment as fd_first_free. This
> > > function is static and the only caller always passes 0 as low. Given
> > > that, how about the following:
> > > http://people.freebsd.org/~mjg/patches/fd_last_used-cleanup.patch
> >
> > Looks good too.
> >
>
> Updated in similar manner:
> http://people.freebsd.org/~mjg/patches/fd_last_used-cleanup2.patch
Ok.
--
Pawel Jakub Dawidek http://www.wheelsystems.com
FreeBSD committer http://www.FreeBSD.org
Am I Evil? Yes, I Am! http://tupytaj.pl
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/svn-src-all/attachments/20120612/f7e6b10b/attachment-0001.pgp
More information about the svn-src-all
mailing list