On Thu, May 31, 2018 at 11:23:55AM -0700, Dennis Dalessandro wrote: > From: Mike Marciniszyn <mike.marciniszyn@xxxxxxxxx> > > This field should be in ctxtdata to allow for > better locality of access by eliminating a dd > dereference. > > The new field is now side-by-side with rcvhdrqentsize > since the rhf_offset is a function of the rcvhdrqentsize. > > Both fields are now correctly sized as u8. > int hfi1_reset_device(int); > diff --git a/drivers/infiniband/hw/hfi1/init.c b/drivers/infiniband/hw/hfi1/init.c > index c71659e..a203a68 100644 > +++ b/drivers/infiniband/hw/hfi1/init.c > @@ -113,8 +113,8 @@ > MODULE_PARM_DESC(rcvhdrcnt, "Receive header queue count (default 2048)"); > > static uint hfi1_hdrq_entsize = 32; > -module_param_named(hdrq_entsize, hfi1_hdrq_entsize, uint, S_IRUGO); > -MODULE_PARM_DESC(hdrq_entsize, "Size of header queue entries: 2 - 8B, 16 - 64B (default), 32 - 128B"); > +module_param_named(hdrq_entsize, hfi1_hdrq_entsize, uint, 0444); > +MODULE_PARM_DESC(hdrq_entsize, "Size of header queue entries: 2 - 8B, 16 - 64B, 32 - 128B (default)"); How is this hunk related to this patch? Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html