static void ceph_fault(struct ceph_connection *con)
           __releases(con->mutex)
   {
           pr_err("%s%lld %s %s\n", ENTITY_NAME(con->peer_name),
                  ceph_pr_addr(&con->peer_addr.in_addr), con->error_msg)

Perhaps this should become pr_info() or something.  Sage?

Yeah, I think pr_info() is probably the right choice.  Do you know if that
hits the console by default, or just dmesg/kern.log?

pr_info is level 6, KERN_INFO; by default, /proc/sys/kernel/printk has
4 4 1 7 in it, the first 4 of which means 4-and-lower go to console.  So
debug, info, notice messages all are options for "not console".

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
  • Re: Wido den Hollander
    • Re: jie sun
      • Re: Gregory Farnum
        • Re: Gregory Farnum
          • Re: Alex Elder
            • Re: Sage Weil
              • Re: Dan Mick
                • Re: Alex Elder
          • Re: jie sun

Reply via email to