Re: [PATCH V2] block: sysfs fix mismatched queue_var_{store,show}in 64bit kernel

From: Tejun Heo
Date: Fri Jul 17 2009 - 03:53:49 EST


Hello,

Danny Feng wrote:
> Thank you very much for the review, I've attached v3 patch follow
> your advices, please help me review.

Thanks. I queued the patch on the following temp block tree.

git://git.kernel.org/pub/scm/linux/kernel/git/tj/misc.git tj-block-for-linus

> Another thing on block sysfs is that we can echo some invalid value
> (e.g 123abc), should sysfs return -EINVAL for those?

sysfs nodes are only accessible from root and it's customary to not
invest too much into safe guarding the input. So, leaving as it is
should be fine, I think.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/