> > > + req = (struct drm_i915_gem_request *) > > > + kmalloc(sizeof(struct drm_i915_gem_request), GFP_KERNEL); > > > + req->ring = ring; > > > + req->ctx = to; > > > + i915_gem_context_reference(req->ctx); > > > + req->tail = tail; > > > > Need to test if the allocation has succeeded and return an error if not. > > Also, no need for the cast, kmalloc returns a void *. > > We usually use sizeof(*req) to be safe against variables changing type without > updating the allocation size (someont the compiler wouldn't warn about). Ok, will do. _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
