[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