Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Feedback-ID: idfb84289:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-372-g43825cb665-fm-20230411.003-g43825cb6
Mime-Version: 1.0
Message-Id: <996b59f0-ea87-4008-a908-35b29b550b1b@app.fastmail.com>
In-Reply-To: <ZEEr0r/6/lLiwlsy@minyard.net>
References: <20230419150032.3483229-1-govert.overgaauw@prodrive-technologies.com>
 <ZEEr0r/6/lLiwlsy@minyard.net>
Date:   Fri, 21 Apr 2023 11:28:05 +0930
From:   "Andrew Jeffery" <andrew@aj.id.au>
To:     "Corey Minyard" <minyard@acm.org>,
        "Govert Overgaauw" <govert.overgaauw@prodrive-technologies.com>,
        vernon.mauery@linux.intel.com
Cc:     openipmi-developer@lists.sourceforge.net,
        linux-kernel@vger.kernel.org, "Zev Weiss" <zev@bewilderbeest.net>
Subject: Re: [Openipmi-developer] [PATCH] ipmi: ipmi-bmc: Improve errno returned to
 userspace
Content-Type: text/plain
Precedence: bulk

Hi Corey,

On Thu, 20 Apr 2023, at 21:40, Corey Minyard wrote:
> Andrew, what do you think?

I'm a bit short on details of the IPMI KCS state machine and expectations of userspace in this exact case.

I've added Vernon who is one of the IPMI maintainers for OpenBMC.

I've also added Zev who's an interested party, and openbmc@ for good measure.

Andrew

>
> -corey
>
> On Wed, Apr 19, 2023 at 05:00:32PM +0200, Govert Overgaauw via 
> Openipmi-developer wrote:
>> While the KCS driver is not in KCS_PHASE_WAIT_READ state it returns
>> -EINVAL to userspace on a write call. change this to -EAGAIN to indicate
>> that the error is related to the state and not the argument.
>> 
>> Signed-off-by: Govert Overgaauw <govert.overgaauw@prodrive-technologies.com>
>> ---
>>  drivers/char/ipmi/kcs_bmc_cdev_ipmi.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/drivers/char/ipmi/kcs_bmc_cdev_ipmi.c b/drivers/char/ipmi/kcs_bmc_cdev_ipmi.c
>> index cf670e891966..4c7400faf333 100644
>> --- a/drivers/char/ipmi/kcs_bmc_cdev_ipmi.c
>> +++ b/drivers/char/ipmi/kcs_bmc_cdev_ipmi.c
>> @@ -405,7 +405,7 @@ static ssize_t kcs_bmc_ipmi_write(struct file *filp, const char __user *buf,
>>  		kcs_bmc_write_data(priv->client.dev, priv->data_out[0]);
>>  		ret = count;
>>  	} else {
>> -		ret = -EINVAL;
>> +		ret = -EAGAIN;
>>  	}
>>  	spin_unlock_irq(&priv->lock);
>>  
>> -- 
>> 2.30.2
>> 
>> 
>> 
>> _______________________________________________
>> Openipmi-developer mailing list
>> Openipmi-developer@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/openipmi-developer