[ofa-general][PATCH] mlx4_core: Multi Protocol support

Roland Dreier rdreier at cisco.com
Wed Apr 16 11:49:22 PDT 2008


You have

 > +static struct mlx4_db_pgdir *mlx4_alloc_db_pgdir(struct device *dma_device)
 > +{
 > +	struct mlx4_db_pgdir *pgdir;
 > +
 > +	pgdir = kzalloc(sizeof *pgdir, GFP_KERNEL);
 > +	if (!pgdir)
 > +		return NULL;
 > +
 > +	bitmap_fill(pgdir->order1, MLX4_DB_PER_PAGE / 2);

and so on...

If you're going to move the doorbell stuff from mlx4_ib to mlx4_core,
that's fine, but really move it: you should remove the code from mlx4_ib
and use the stuff in mlx4_core rather than having the same stuff
duplicated in two places.  Especially since as this patch stands now,
there are *no* users for the doorbell code in mlx4_core.

 - R.



More information about the general mailing list