Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1232740

Re: [PATCH] hid: enable hid device to suspend/resume asynchronously

From Jiri Kosina <jikos@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH] hid: enable hid device to suspend/resume asynchronously
Date 2015-09-25 14:10 +0200
Message-ID <qczZ8-T1-11@gated-at.bofh.it> (permalink)
References <pYuZk-3nQ-13@gated-at.bofh.it> <q4Z4o-39P-63@gated-at.bofh.it> <qc7Tc-2Wj-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Thu, 24 Sep 2015, Fu, Zhonghui wrote:

> >> Enable hid device to suspend/resume asynchronously. This can improve
> >> system suspend/resume speed.
> > How well was this tested?
> >
> > Power management is notorious for not being really in excellent shape on 
> > many HID devices.
> >
> > So I'd like to be careful.
> I were on leave these days, so sorry for late reply.
> 
> This can reduce system suspend/resume time about 20ms, from 1030ms to 
> 1010ms on ASUS T100TA machine. Although the improvement is not very 
> significant, but this can let your parent/child device suspend/resume 
> more asynchronously and take advantage of multicore to improve overall 
> system suspend/resume speed.
> 
> I have resent this patch with updated commit message - "[PATCH v2] hid: 
> enable hid device to suspend/resume asynchronously".

Is there any reason why not enable this from userspace via sysfs instead, 
based on whitelist on known-to-be-well-behaved devices, from udev?

I am really careful when it comes to enabling features like this 
(especially given the state of PM support on some low-end devices) 
globally.

Thanks,

-- 
Jiri Kosina
SUSE Labs

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

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

Re: [PATCH] hid: enable hid device to suspend/resume asynchronously "Fu, Zhonghui" <zhonghui.fu@linux.intel.com> - 2015-09-24 08:10 +0200
  Re: [PATCH] hid: enable hid device to suspend/resume  asynchronously Jiri Kosina <jikos@kernel.org> - 2015-09-25 14:10 +0200

csiph-web