Re: [PATCH rdma-next v3 1/7] RDMA/restrack: Add general infrastructure to track RDMA resources

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

 



On Thu, Jan 11, 2018 at 07:47:27PM +0000, Bart Van Assche wrote:
> On Thu, 2018-01-11 at 17:31 +0200, Leon Romanovsky wrote:
> > +struct rdma_restrack_entry {
> > +       struct list_head        list;
> > +
> > +       /*
> > +        * The entries are filled during rdma_restrack_add,
> > +        * can be attempted to be free during rdma_restrack_del.
> > +        *
> > +        * As an example for that, see mlx5 QPs with type MLX5_IB_QPT_HW_GSI
> > +        */
> > +       bool                    valid;
> > +
> > +       /*
> > +        * Sleepabale RCU to protect object data.
> > +        */
> > +       struct srcu_struct      srcu;
> > +
> > +       struct task_struct      *task;
> > +       char                    *task_comm;
> > +};
>
> Please use the kernel-doc syntax to document the meaning of the names of the
> fields in this structure. It is e.g. nontrivial to guess what "task_comm" stands
> for. The presense of a task_struct pointer is also nontrivial.
>
> PS: I think the spelling of "Sleepabale" should be fixed.

I'll fix and resend.

>
> Thanks,
>
> Bart.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux